[InstrProf] Change step from 64-bit to pointer-sized int

jdmitrovic-syrmia · jdmitrovic-syrmia · commit a45c783d0686 · 2024-02-28T10:43:46.000+01:00
Fixed 64-bit step can lead to creating atomic instructions unsupported by the target architecture (see rust-lang/rust#112313).
diff --git a/llvm/lib/IR/IntrinsicInst.cpp b/llvm/lib/IR/IntrinsicInst.cpp
@@ -288,7 +288,8 @@ Value *InstrProfIncrementInst::getStep() const {
   }
   const Module *M = getModule();
   LLVMContext &Context = M->getContext();
-  return ConstantInt::get(Type::getInt64Ty(Context), 1);
+  const auto &DL = M->getDataLayout();
+  return ConstantInt::get(DL.getIntPtrType(Context), 1);
 }
 
 std::optional<RoundingMode> ConstrainedFPIntrinsic::getRoundingMode() const {

Original file line number	Diff line number	Diff line change
`@@ -288,7 +288,8 @@ Value *InstrProfIncrementInst::getStep() const {`
`288`	`288`	`}`
`289`	`289`	`const Module *M = getModule();`
`290`	`290`	`LLVMContext &Context = M->getContext();`
`291`		`- return ConstantInt::get(Type::getInt64Ty(Context), 1);`
	`291`	`+ const auto &DL = M->getDataLayout();`
	`292`	`+ return ConstantInt::get(DL.getIntPtrType(Context), 1);`
`292`	`293`	`}`
`293`	`294`
`294`	`295`	`std::optional<RoundingMode> ConstrainedFPIntrinsic::getRoundingMode() const {`