[llvm] 259fa70 - [PGO] incorrect classof in InstrProfIncrementInst
Ellis Hoag via llvm-commits
llvm-commits at lists.llvm.org
Fri Jan 20 10:22:28 PST 2023
Author: Ellis Hoag
Date: 2023-01-20T10:22:22-08:00
New Revision: 259fa70885e9e0e9e396b198829769276a6a6069
URL: https://github.com/llvm/llvm-project/commit/259fa70885e9e0e9e396b198829769276a6a6069
DIFF: https://github.com/llvm/llvm-project/commit/259fa70885e9e0e9e396b198829769276a6a6069.diff
LOG: [PGO] incorrect classof in InstrProfIncrementInst
Class InstrProfIncrementInstStep inherits from InstrProfIncrementInst but cannot cast to InstrProfIncrementInst, because InstrProfIncrementInst::classof does not cover such circumstance。
Function InstrProfiling::run traverse all instruction in a module and try to cast them to InstrProfIncrementInst using dyn_cast, but it will return nullptr if the instruction is InstrProfIncrementInstStep(subclass of InstrProfIncrementInst).
Reviewed By: tejohnson, ellis
Differential Revision: https://reviews.llvm.org/D141579
Added:
llvm/test/Instrumentation/InstrProfiling/before-value-profile-lowering.ll
Modified:
llvm/include/llvm/IR/IntrinsicInst.h
Removed:
################################################################################
diff --git a/llvm/include/llvm/IR/IntrinsicInst.h b/llvm/include/llvm/IR/IntrinsicInst.h
index ea392ba574bae..df6ce27d1bbdd 100644
--- a/llvm/include/llvm/IR/IntrinsicInst.h
+++ b/llvm/include/llvm/IR/IntrinsicInst.h
@@ -1355,7 +1355,8 @@ class InstrProfCoverInst : public InstrProfInstBase {
class InstrProfIncrementInst : public InstrProfInstBase {
public:
static bool classof(const IntrinsicInst *I) {
- return I->getIntrinsicID() == Intrinsic::instrprof_increment;
+ return I->getIntrinsicID() == Intrinsic::instrprof_increment ||
+ I->getIntrinsicID() == Intrinsic::instrprof_increment_step;
}
static bool classof(const Value *V) {
return isa<IntrinsicInst>(V) && classof(cast<IntrinsicInst>(V));
diff --git a/llvm/test/Instrumentation/InstrProfiling/before-value-profile-lowering.ll b/llvm/test/Instrumentation/InstrProfiling/before-value-profile-lowering.ll
new file mode 100644
index 0000000000000..5dfec433f4ecb
--- /dev/null
+++ b/llvm/test/Instrumentation/InstrProfiling/before-value-profile-lowering.ll
@@ -0,0 +1,23 @@
+; Test to ensure that when the only instrprof increment instruction is
+; increment by step instruction and there is value profile instruction
+; in front of all increment instructions in a function,
+; the profile data variable is generated before value profile lowering
+
+; RUN: opt < %s -passes=instrprof -S | FileCheck %s
+
+target triple = "x86_64-unknown-linux-gnu"
+
+declare void @llvm.instrprof.increment.step(i8*, i64, i32, i32, i64)
+
+declare void @llvm.instrprof.value.profile(i8*, i64, i64, i32, i32)
+
+; CHECK: @__profd_foo = private global
+ at __profn_foo = private constant [3 x i8] c"foo"
+
+define i32 @foo(i32 ()* ) {
+ %2 = ptrtoint i32 ()* %0 to i64
+ call void @llvm.instrprof.value.profile(i8* getelementptr inbounds ([3 x i8], [3 x i8]* @__profn_foo, i32 0, i32 0), i64 0, i64 %2, i32 0, i32 0)
+ call void @llvm.instrprof.increment.step(i8* getelementptr inbounds ([3 x i8], [3 x i8]* @__profn_foo, i32 0, i32 0), i64 0, i32 1, i32 0, i64 0)
+ %3 = tail call i32 %0()
+ ret i32 %3
+}
More information about the llvm-commits
mailing list