diff options
| author | Brian Gesiak <modocache@gmail.com> | 2017-06-30 18:13:59 +0000 |
|---|---|---|
| committer | Brian Gesiak <modocache@gmail.com> | 2017-06-30 18:13:59 +0000 |
| commit | 44e5f6c4acb40c7330686477d7c61a1ffe17029f (patch) | |
| tree | a84c765c5534485ace6b2e10d3daebb90f3ba7d2 /llvm/tools/opt/opt.cpp | |
| parent | 724990ab643b5497bfa3b3350c11dacbc9d9109b (diff) | |
| download | bcm5719-llvm-44e5f6c4acb40c7330686477d7c61a1ffe17029f.tar.gz bcm5719-llvm-44e5f6c4acb40c7330686477d7c61a1ffe17029f.zip | |
[ORE] Unify spelling as "diagnostics hotness"
Summary:
To enable profile hotness information in diagnostics output, Clang takes
the option `-fdiagnostics-show-hotness` -- that's "diagnostics", with an
"s" at the end. Clang also defines `CodeGenOptions::DiagnosticsWithHotness`.
LLVM, on the other hand, defines
`LLVMContext::getDiagnosticHotnessRequested` -- that's "diagnostic", not
"diagnostics". It's a small difference, but it's confusing, typo-inducing, and
frustrating.
Add a new method with the spelling "diagnostics", and "deprecate" the
old spelling.
Reviewers: anemet, davidxl
Reviewed By: anemet
Subscribers: llvm-commits, mehdi_amini
Differential Revision: https://reviews.llvm.org/D34864
llvm-svn: 306848
Diffstat (limited to 'llvm/tools/opt/opt.cpp')
| -rw-r--r-- | llvm/tools/opt/opt.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/llvm/tools/opt/opt.cpp b/llvm/tools/opt/opt.cpp index 9d489ab5a2d..480deac0201 100644 --- a/llvm/tools/opt/opt.cpp +++ b/llvm/tools/opt/opt.cpp @@ -420,7 +420,7 @@ int main(int argc, char **argv) { Context.enableDebugTypeODRUniquing(); if (PassRemarksWithHotness) - Context.setDiagnosticHotnessRequested(true); + Context.setDiagnosticsHotnessRequested(true); std::unique_ptr<tool_output_file> YamlFile; if (RemarksFilename != "") { |
