[PATCH] D34864: [ORE] Unify spelling as "diagnostics hotness"
Brian Gesiak via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Jun 29 23:32:02 PDT 2017
modocache created this revision.
Herald added a subscriber: mehdi_amini.
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.
https://reviews.llvm.org/D34864
Files:
include/llvm/Analysis/OptimizationDiagnosticInfo.h
include/llvm/CodeGen/MachineOptimizationRemarkEmitter.h
include/llvm/IR/LLVMContext.h
lib/Analysis/OptimizationDiagnosticInfo.cpp
lib/CodeGen/MachineOptimizationRemarkEmitter.cpp
lib/IR/LLVMContext.cpp
lib/IR/LLVMContextImpl.h
lib/LTO/LTO.cpp
tools/llc/llc.cpp
tools/opt/opt.cpp
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D34864.104818.patch
Type: text/x-patch
Size: 7576 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170630/8f9a961a/attachment.bin>
More information about the llvm-commits
mailing list