Skip to content

Commit 694c52c

Browse files
authored
[llvm][misexpect][NFC] Fix Twine usage (#136446)
1 parent 11d5c4d commit 694c52c

File tree

3 files changed

+3
-4
lines changed

3 files changed

+3
-4
lines changed

llvm/include/llvm/IR/DiagnosticInfo.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1090,7 +1090,7 @@ class DiagnosticInfoUnsupported : public DiagnosticInfoWithLocationBase {
10901090
/// Diagnostic information for MisExpect analysis.
10911091
class DiagnosticInfoMisExpect : public DiagnosticInfoWithLocationBase {
10921092
public:
1093-
DiagnosticInfoMisExpect(const Instruction *Inst, Twine &Msg);
1093+
DiagnosticInfoMisExpect(const Instruction *Inst, const Twine &Msg);
10941094

10951095
/// \see DiagnosticInfo::print.
10961096
void print(DiagnosticPrinter &DP) const override;

llvm/lib/IR/DiagnosticInfo.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -443,7 +443,7 @@ std::string DiagnosticInfoOptimizationBase::getMsg() const {
443443
}
444444

445445
DiagnosticInfoMisExpect::DiagnosticInfoMisExpect(const Instruction *Inst,
446-
Twine &Msg)
446+
const Twine &Msg)
447447
: DiagnosticInfoWithLocationBase(DK_MisExpect, DS_Warning,
448448
*Inst->getParent()->getParent(),
449449
Inst->getDebugLoc()),

llvm/lib/Transforms/Utils/MisExpect.cpp

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -106,10 +106,9 @@ void emitMisexpectDiagnostic(Instruction *I, LLVMContext &Ctx,
106106
"Potential performance regression from use of the llvm.expect intrinsic: "
107107
"Annotation was correct on {0} of profiled executions.",
108108
PerString);
109-
Twine Msg(PerString);
110109
Instruction *Cond = getInstCondition(I);
111110
if (isMisExpectDiagEnabled(Ctx))
112-
Ctx.diagnose(DiagnosticInfoMisExpect(Cond, Msg));
111+
Ctx.diagnose(DiagnosticInfoMisExpect(Cond, Twine(PerString)));
113112
OptimizationRemarkEmitter ORE(I->getParent()->getParent());
114113
ORE.emit(OptimizationRemark(DEBUG_TYPE, "misexpect", Cond) << RemStr.str());
115114
}

0 commit comments

Comments
 (0)