diff options
| author | Roman Lebedev <lebedev.ri@gmail.com> | 2020-11-14 09:44:14 +0300 |
|---|---|---|
| committer | Roman Lebedev <lebedev.ri@gmail.com> | 2020-11-14 13:12:38 +0300 |
| commit | 6861d938e5c946cc7079d9849ef7560d07aa2d80 (patch) | |
| tree | 0134b8f98b411e7435f8f4f0a58851fa3724750e /clang/lib/Frontend/CompilerInvocation.cpp | |
| parent | e51631ca4cf461510019d5fc950eacb2fe45eb48 (diff) | |
| download | llvm-6861d938e5c946cc7079d9849ef7560d07aa2d80.zip llvm-6861d938e5c946cc7079d9849ef7560d07aa2d80.tar.gz llvm-6861d938e5c946cc7079d9849ef7560d07aa2d80.tar.bz2 | |
Revert "clang-misexpect: Profile Guided Validation of Performance Annotations in LLVM"
See discussion in https://bugs.llvm.org/show_bug.cgi?id=45073 / https://reviews.llvm.org/D66324#2334485
the implementation is known-broken for certain inputs,
the bugreport was up for a significant amount of timer,
and there has been no activity to address it.
Therefore, just completely rip out all of misexpect handling.
I suspect, fixing it requires redesigning the internals of MD_misexpect.
Should anyone commit to fixing the implementation problem,
starting from clean slate may be better anyways.
This reverts commit 7bdad08429411e7d0ecd58cd696b1efe3cff309e,
and some of it's follow-ups, that don't stand on their own.
Diffstat (limited to 'clang/lib/Frontend/CompilerInvocation.cpp')
| -rw-r--r-- | clang/lib/Frontend/CompilerInvocation.cpp | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/clang/lib/Frontend/CompilerInvocation.cpp b/clang/lib/Frontend/CompilerInvocation.cpp index ba6ff82..2e20ece 100644 --- a/clang/lib/Frontend/CompilerInvocation.cpp +++ b/clang/lib/Frontend/CompilerInvocation.cpp @@ -3833,9 +3833,6 @@ bool CompilerInvocation::CreateFromArgs(CompilerInvocation &Res, } } - if (Diags.isIgnored(diag::warn_profile_data_misexpect, SourceLocation())) - Res.FrontendOpts.LLVMArgs.push_back("-pgo-warn-misexpect"); - LangOpts.FunctionAlignment = getLastArgIntValue(Args, OPT_function_alignment, 0, Diags); |