Skip to content

Commit ec559c4

Browse files
jayfoadjayfoad
committed
[LLVM] Make more use of IRBuilder::CreateIntrinsic. NFC.
Convert almost every instance of: CreateCall(Intrinsic::getOrInsertDeclaration(...), ...) to the equivalent CreateIntrinsic call.
1 parent 157f10d commit ec559c4

File tree

13 files changed

+101
-162
lines changed

13 files changed

+101
-162
lines changed

‎llvm/lib/CodeGen/SafeStack.cpp‎

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -368,8 +368,7 @@ Value *SafeStack::getStackGuard(IRBuilder<> &IRB, Function &F) {
368368

369369
if (!StackGuardVar) {
370370
TL.insertSSPDeclarations(*M);
371-
return IRB.CreateCall(
372-
Intrinsic::getOrInsertDeclaration(M, Intrinsic::stackguard));
371+
return IRB.CreateIntrinsic(Intrinsic::stackguard, {}, {});
373372
}
374373

375374
return IRB.CreateLoad(StackPtrTy, StackGuardVar, "StackGuard");

‎llvm/lib/CodeGen/StackProtector.cpp‎

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -519,8 +519,7 @@ static Value *getStackGuard(const TargetLoweringBase *TLI, Module *M,
519519
if (SupportsSelectionDAGSP)
520520
*SupportsSelectionDAGSP = true;
521521
TLI->insertSSPDeclarations(*M);
522-
return B.CreateCall(
523-
Intrinsic::getOrInsertDeclaration(M, Intrinsic::stackguard));
522+
return B.CreateIntrinsic(Intrinsic::stackguard, {}, {});
524523
}
525524

526525
/// Insert code into the entry block that stores the stack guard
@@ -541,8 +540,7 @@ static bool CreatePrologue(Function *F, Module *M, Instruction *CheckLoc,
541540
AI = B.CreateAlloca(PtrTy, nullptr, "StackGuardSlot");
542541

543542
Value *GuardSlot = getStackGuard(TLI, M, B, &SupportsSelectionDAGSP);
544-
B.CreateCall(Intrinsic::getOrInsertDeclaration(M, Intrinsic::stackprotector),
545-
{GuardSlot, AI});
543+
B.CreateIntrinsic(Intrinsic::stackprotector, {}, {GuardSlot, AI});
546544
return SupportsSelectionDAGSP;
547545
}
548546

‎llvm/lib/IR/AutoUpgrade.cpp‎

Lines changed: 55 additions & 95 deletions
Large diffs are not rendered by default.

‎llvm/lib/Target/AArch64/AArch64ISelLowering.cpp‎

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -27284,8 +27284,7 @@ Value *AArch64TargetLowering::emitLoadLinked(IRBuilderBase &Builder,
2728427284
void AArch64TargetLowering::emitAtomicCmpXchgNoStoreLLBalance(
2728527285
IRBuilderBase &Builder) const {
2728627286
Module *M = Builder.GetInsertBlock()->getParent()->getParent();
27287-
Builder.CreateCall(
27288-
Intrinsic::getOrInsertDeclaration(M, Intrinsic::aarch64_clrex));
27287+
Builder.CreateIntrinsic(Intrinsic::aarch64_clrex, {}, {});
2728927288
}
2729027289

2729127290
Value *AArch64TargetLowering::emitStoreConditional(IRBuilderBase &Builder,

‎llvm/lib/Target/AMDGPU/AMDGPUSwLowerLDS.cpp‎

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -921,9 +921,8 @@ void AMDGPUSwLowerLDS::lowerKernelLDSAccesses(Function *Func,
921921
FunctionCallee AsanFreeFunc = M.getOrInsertFunction(
922922
StringRef("__asan_free_impl"),
923923
FunctionType::get(IRB.getVoidTy(), {Int64Ty, Int64Ty}, false));
924-
Value *ReturnAddr = IRB.CreateCall(
925-
Intrinsic::getOrInsertDeclaration(&M, Intrinsic::returnaddress),
926-
IRB.getInt32(0));
924+
Value *ReturnAddr =
925+
IRB.CreateIntrinsic(Intrinsic::returnaddress, {}, IRB.getInt32(0));
927926
Value *RAPToInt = IRB.CreatePtrToInt(ReturnAddr, Int64Ty);
928927
Value *MallocPtrToInt = IRB.CreatePtrToInt(LoadMallocPtr, Int64Ty);
929928
IRB.CreateCall(AsanFreeFunc, {MallocPtrToInt, RAPToInt});

‎llvm/lib/Target/ARM/ARMISelLowering.cpp‎

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -21446,8 +21446,7 @@ void ARMTargetLowering::emitAtomicCmpXchgNoStoreLLBalance(
2144621446
if (!Subtarget->hasV7Ops())
2144721447
return;
2144821448
Module *M = Builder.GetInsertBlock()->getParent()->getParent();
21449-
Builder.CreateCall(
21450-
Intrinsic::getOrInsertDeclaration(M, Intrinsic::arm_clrex));
21449+
Builder.CreateIntrinsic(Intrinsic::arm_clrex, {}, {});
2145121450
}
2145221451

2145321452
Value *ARMTargetLowering::emitStoreConditional(IRBuilderBase &Builder,

‎llvm/lib/Target/PowerPC/PPCISelLowering.cpp‎

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -12205,11 +12205,8 @@ Instruction *PPCTargetLowering::emitTrailingFence(IRBuilderBase &Builder,
1220512205
// http://www.rdrop.com/users/paulmck/scalability/paper/N2745r.2011.03.04a.html
1220612206
// and http://www.cl.cam.ac.uk/~pes20/cppppc/ for justification.
1220712207
if (isa<LoadInst>(Inst))
12208-
return Builder.CreateCall(
12209-
Intrinsic::getOrInsertDeclaration(
12210-
Builder.GetInsertBlock()->getParent()->getParent(),
12211-
Intrinsic::ppc_cfence, {Inst->getType()}),
12212-
{Inst});
12208+
return Builder.CreateIntrinsic(Intrinsic::ppc_cfence, {Inst->getType()},
12209+
{Inst});
1221312210
// FIXME: Can use isync for rmw operation.
1221412211
return callIntrinsic(Builder, Intrinsic::ppc_lwsync);
1221512212
}

‎llvm/lib/Target/X86/X86WinEHState.cpp‎

Lines changed: 7 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -333,12 +333,10 @@ void WinEHStatePass::emitExceptionRegistrationRecord(Function *F) {
333333
// If using _except_handler4, the EHGuard contains: FramePtr xor Cookie.
334334
if (UseStackGuard) {
335335
Value *Val = Builder.CreateLoad(Int32Ty, Cookie);
336-
Value *FrameAddr = Builder.CreateCall(
337-
Intrinsic::getOrInsertDeclaration(
338-
TheModule, Intrinsic::frameaddress,
339-
Builder.getPtrTy(
340-
TheModule->getDataLayout().getAllocaAddrSpace())),
341-
Builder.getInt32(0), "frameaddr");
336+
Value *FrameAddr = Builder.CreateIntrinsic(
337+
Intrinsic::frameaddress,
338+
Builder.getPtrTy(TheModule->getDataLayout().getAllocaAddrSpace()),
339+
Builder.getInt32(0), /*FMFSource=*/nullptr, "frameaddr");
342340
Value *FrameAddrI32 = Builder.CreatePtrToInt(FrameAddr, Int32Ty);
343341
FrameAddrI32 = Builder.CreateXor(FrameAddrI32, Val);
344342
Builder.CreateStore(FrameAddrI32, EHGuardNode);
@@ -369,8 +367,7 @@ void WinEHStatePass::emitExceptionRegistrationRecord(Function *F) {
369367
}
370368

371369
Value *WinEHStatePass::emitEHLSDA(IRBuilder<> &Builder, Function *F) {
372-
return Builder.CreateCall(
373-
Intrinsic::getOrInsertDeclaration(TheModule, Intrinsic::x86_seh_lsda), F);
370+
return Builder.CreateIntrinsic(Intrinsic::x86_seh_lsda, {}, F);
374371
}
375372

376373
/// Generate a thunk that puts the LSDA of ParentFunc in EAX and then calls
@@ -624,17 +621,13 @@ void WinEHStatePass::addStateStores(Function &F, WinEHFuncInfo &FuncInfo) {
624621
// that it can recover the original frame pointer.
625622
IRBuilder<> Builder(RegNode->getNextNode());
626623
Value *RegNodeI8 = Builder.CreateBitCast(RegNode, Builder.getPtrTy());
627-
Builder.CreateCall(Intrinsic::getOrInsertDeclaration(
628-
TheModule, Intrinsic::x86_seh_ehregnode),
629-
{RegNodeI8});
624+
Builder.CreateIntrinsic(Intrinsic::x86_seh_ehregnode, {}, {RegNodeI8});
630625

631626
if (EHGuardNode) {
632627
IRBuilder<> Builder(EHGuardNode->getNextNode());
633628
Value *EHGuardNodeI8 =
634629
Builder.CreateBitCast(EHGuardNode, Builder.getPtrTy());
635-
Builder.CreateCall(Intrinsic::getOrInsertDeclaration(
636-
TheModule, Intrinsic::x86_seh_ehguard),
637-
{EHGuardNodeI8});
630+
Builder.CreateIntrinsic(Intrinsic::x86_seh_ehguard, {}, {EHGuardNodeI8});
638631
}
639632

640633
// Calculate state numbers.

‎llvm/lib/Transforms/Instrumentation/AddressSanitizer.cpp‎

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1866,10 +1866,9 @@ void AddressSanitizer::instrumentAddress(Instruction *OrigIns,
18661866
if (UseCalls && ClOptimizeCallbacks) {
18671867
const ASanAccessInfo AccessInfo(IsWrite, CompileKernel, AccessSizeIndex);
18681868
Module *M = IRB.GetInsertBlock()->getParent()->getParent();
1869-
IRB.CreateCall(
1870-
Intrinsic::getOrInsertDeclaration(M, Intrinsic::asan_check_memaccess),
1871-
{IRB.CreatePointerCast(Addr, PtrTy),
1872-
ConstantInt::get(Int32Ty, AccessInfo.Packed)});
1869+
IRB.CreateIntrinsic(Intrinsic::asan_check_memaccess, {},
1870+
{IRB.CreatePointerCast(Addr, PtrTy),
1871+
ConstantInt::get(Int32Ty, AccessInfo.Packed)});
18731872
return;
18741873
}
18751874

‎llvm/lib/Transforms/Instrumentation/HWAddressSanitizer.cpp‎

Lines changed: 9 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -1041,19 +1041,18 @@ void HWAddressSanitizer::instrumentMemAccessOutline(Value *Ptr, bool IsWrite,
10411041
}
10421042

10431043
if (UseFixedShadowIntrinsic) {
1044-
IRB.CreateCall(
1045-
Intrinsic::getOrInsertDeclaration(
1046-
M, UseShortGranules
1047-
? Intrinsic::hwasan_check_memaccess_shortgranules_fixedshadow
1048-
: Intrinsic::hwasan_check_memaccess_fixedshadow),
1044+
IRB.CreateIntrinsic(
1045+
UseShortGranules
1046+
? Intrinsic::hwasan_check_memaccess_shortgranules_fixedshadow
1047+
: Intrinsic::hwasan_check_memaccess_fixedshadow,
1048+
{},
10491049
{Ptr, ConstantInt::get(Int32Ty, AccessInfo),
10501050
ConstantInt::get(Int64Ty, Mapping.offset())});
10511051
} else {
1052-
IRB.CreateCall(Intrinsic::getOrInsertDeclaration(
1053-
M, UseShortGranules
1054-
? Intrinsic::hwasan_check_memaccess_shortgranules
1055-
: Intrinsic::hwasan_check_memaccess),
1056-
{ShadowBase, Ptr, ConstantInt::get(Int32Ty, AccessInfo)});
1052+
IRB.CreateIntrinsic(
1053+
UseShortGranules ? Intrinsic::hwasan_check_memaccess_shortgranules
1054+
: Intrinsic::hwasan_check_memaccess,
1055+
{}, {ShadowBase, Ptr, ConstantInt::get(Int32Ty, AccessInfo)});
10571056
}
10581057
}
10591058

0 commit comments

Comments
 (0)