diff options
author | Youngsuk Kim <youngsuk.kim@hpe.com> | 2023-11-29 09:20:15 -0600 |
---|---|---|
committer | Youngsuk Kim <youngsuk.kim@hpe.com> | 2023-11-29 10:22:31 -0600 |
commit | 859338a695169d2804d6999f44e1e94bedf6ddd7 (patch) | |
tree | 0bad915f3ce4466c0e8b5be657cb58c5479b3a38 | |
parent | 3bf72bf427f15521111df4e35c868005ceb0013b (diff) | |
download | llvm-859338a695169d2804d6999f44e1e94bedf6ddd7.zip llvm-859338a695169d2804d6999f44e1e94bedf6ddd7.tar.gz llvm-859338a695169d2804d6999f44e1e94bedf6ddd7.tar.bz2 |
[llvm] Replace uses of Type::getPointerTo (NFC)
Work towards removing method Type::getPointerTo.
Opaque ptr cleanup effort.
-rw-r--r-- | llvm/lib/CodeGen/SafeStack.cpp | 2 | ||||
-rw-r--r-- | llvm/lib/CodeGen/ShadowStackGCLowering.cpp | 6 | ||||
-rw-r--r-- | llvm/lib/Transforms/Coroutines/CoroInternal.h | 3 | ||||
-rw-r--r-- | llvm/lib/Transforms/Instrumentation/GCOVProfiling.cpp | 13 | ||||
-rw-r--r-- | llvm/lib/Transforms/Vectorize/VectorCombine.cpp | 6 |
5 files changed, 14 insertions, 16 deletions
diff --git a/llvm/lib/CodeGen/SafeStack.cpp b/llvm/lib/CodeGen/SafeStack.cpp index 368e693..ef293fa 100644 --- a/llvm/lib/CodeGen/SafeStack.cpp +++ b/llvm/lib/CodeGen/SafeStack.cpp @@ -793,7 +793,7 @@ bool SafeStack::run() { DILocation::get(SP->getContext(), SP->getScopeLine(), 0, SP)); if (SafeStackUsePointerAddress) { FunctionCallee Fn = F.getParent()->getOrInsertFunction( - "__safestack_pointer_address", StackPtrTy->getPointerTo(0)); + "__safestack_pointer_address", IRB.getPtrTy(0)); UnsafeStackPtr = IRB.CreateCall(Fn); } else { UnsafeStackPtr = TL.getSafeStackPointerLocation(IRB); diff --git a/llvm/lib/CodeGen/ShadowStackGCLowering.cpp b/llvm/lib/CodeGen/ShadowStackGCLowering.cpp index 246e9a5..38f6580 100644 --- a/llvm/lib/CodeGen/ShadowStackGCLowering.cpp +++ b/llvm/lib/CodeGen/ShadowStackGCLowering.cpp @@ -326,7 +326,7 @@ bool ShadowStackGCLowering::runOnFunction(Function &F) { // Initialize the map pointer and load the current head of the shadow stack. Instruction *CurrentHead = - AtEntry.CreateLoad(StackEntryTy->getPointerTo(), Head, "gc_currhead"); + AtEntry.CreateLoad(AtEntry.getPtrTy(), Head, "gc_currhead"); Instruction *EntryMapPtr = CreateGEP(Context, AtEntry, ConcreteStackEntryTy, StackEntry, 0, 1, "gc_frame.map"); AtEntry.CreateStore(FrameMap, EntryMapPtr); @@ -368,8 +368,8 @@ bool ShadowStackGCLowering::runOnFunction(Function &F) { Instruction *EntryNextPtr2 = CreateGEP(Context, *AtExit, ConcreteStackEntryTy, StackEntry, 0, 0, "gc_frame.next"); - Value *SavedHead = AtExit->CreateLoad(StackEntryTy->getPointerTo(), - EntryNextPtr2, "gc_savedhead"); + Value *SavedHead = + AtExit->CreateLoad(AtExit->getPtrTy(), EntryNextPtr2, "gc_savedhead"); AtExit->CreateStore(SavedHead, Head); } diff --git a/llvm/lib/Transforms/Coroutines/CoroInternal.h b/llvm/lib/Transforms/Coroutines/CoroInternal.h index 401b3df..0856c49 100644 --- a/llvm/lib/Transforms/Coroutines/CoroInternal.h +++ b/llvm/lib/Transforms/Coroutines/CoroInternal.h @@ -185,7 +185,8 @@ struct LLVM_LIBRARY_VISIBILITY Shape { switch (ABI) { case coro::ABI::Switch: return FunctionType::get(Type::getVoidTy(FrameTy->getContext()), - FrameTy->getPointerTo(), /*IsVarArg*/false); + PointerType::getUnqual(FrameTy->getContext()), + /*IsVarArg=*/false); case coro::ABI::Retcon: case coro::ABI::RetconOnce: return RetconLowering.ResumePrototype->getFunctionType(); diff --git a/llvm/lib/Transforms/Instrumentation/GCOVProfiling.cpp b/llvm/lib/Transforms/Instrumentation/GCOVProfiling.cpp index b6eb553..1ff0a34 100644 --- a/llvm/lib/Transforms/Instrumentation/GCOVProfiling.cpp +++ b/llvm/lib/Transforms/Instrumentation/GCOVProfiling.cpp @@ -1105,14 +1105,11 @@ Function *GCOVProfiler::insertCounterWriteout( StructType *EmitFunctionCallArgsTy = StructType::create( {Builder.getInt32Ty(), Builder.getInt32Ty(), Builder.getInt32Ty()}, "emit_function_args_ty"); - StructType *EmitArcsCallArgsTy = StructType::create( - {Builder.getInt32Ty(), Builder.getInt64Ty()->getPointerTo()}, - "emit_arcs_args_ty"); - StructType *FileInfoTy = - StructType::create({StartFileCallArgsTy, Builder.getInt32Ty(), - EmitFunctionCallArgsTy->getPointerTo(), - EmitArcsCallArgsTy->getPointerTo()}, - "file_info"); + auto *PtrTy = Builder.getPtrTy(); + StructType *EmitArcsCallArgsTy = + StructType::create({Builder.getInt32Ty(), PtrTy}, "emit_arcs_args_ty"); + StructType *FileInfoTy = StructType::create( + {StartFileCallArgsTy, Builder.getInt32Ty(), PtrTy, PtrTy}, "file_info"); Constant *Zero32 = Builder.getInt32(0); // Build an explicit array of two zeros for use in ConstantExpr GEP building. diff --git a/llvm/lib/Transforms/Vectorize/VectorCombine.cpp b/llvm/lib/Transforms/Vectorize/VectorCombine.cpp index acf2b0e..f386c85 100644 --- a/llvm/lib/Transforms/Vectorize/VectorCombine.cpp +++ b/llvm/lib/Transforms/Vectorize/VectorCombine.cpp @@ -262,8 +262,8 @@ bool VectorCombine::vectorizeLoadInsert(Instruction &I) { // It is safe and potentially profitable to load a vector directly: // inselt undef, load Scalar, 0 --> load VecPtr IRBuilder<> Builder(Load); - Value *CastedPtr = Builder.CreatePointerBitCastOrAddrSpaceCast( - SrcPtr, MinVecTy->getPointerTo(AS)); + Value *CastedPtr = + Builder.CreatePointerBitCastOrAddrSpaceCast(SrcPtr, Builder.getPtrTy(AS)); Value *VecLd = Builder.CreateAlignedLoad(MinVecTy, CastedPtr, Alignment); VecLd = Builder.CreateShuffleVector(VecLd, Mask); @@ -325,7 +325,7 @@ bool VectorCombine::widenSubvectorLoad(Instruction &I) { IRBuilder<> Builder(Load); Value *CastedPtr = - Builder.CreatePointerBitCastOrAddrSpaceCast(SrcPtr, Ty->getPointerTo(AS)); + Builder.CreatePointerBitCastOrAddrSpaceCast(SrcPtr, Builder.getPtrTy(AS)); Value *VecLd = Builder.CreateAlignedLoad(Ty, CastedPtr, Alignment); replaceValue(I, *VecLd); ++NumVecLoad; |