[llvm] dd3184c - [unittest,examples] Replace uses of IRBuilder::getInt8PtrTy with getPtrTy. NFC

Fangrui Song via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 27 08:29:19 PST 2023


Author: Fangrui Song
Date: 2023-11-27T08:29:13-08:00
New Revision: dd3184c30ff531b8aecea280e65233337dd02815

URL: https://github.com/llvm/llvm-project/commit/dd3184c30ff531b8aecea280e65233337dd02815
DIFF: https://github.com/llvm/llvm-project/commit/dd3184c30ff531b8aecea280e65233337dd02815.diff

LOG: [unittest,examples] Replace uses of IRBuilder::getInt8PtrTy with getPtrTy. NFC

Added: 
    

Modified: 
    llvm/examples/ExceptionDemo/ExceptionDemo.cpp
    llvm/unittests/Analysis/MemorySSATest.cpp
    llvm/unittests/Frontend/OpenMPIRBuilderTest.cpp
    llvm/unittests/IR/InstructionsTest.cpp
    llvm/unittests/Transforms/Utils/FunctionComparatorTest.cpp

Removed: 
    


################################################################################
diff  --git a/llvm/examples/ExceptionDemo/ExceptionDemo.cpp b/llvm/examples/ExceptionDemo/ExceptionDemo.cpp
index 1b3ec7c91ddee10..0afc6b30d140e98 100644
--- a/llvm/examples/ExceptionDemo/ExceptionDemo.cpp
+++ b/llvm/examples/ExceptionDemo/ExceptionDemo.cpp
@@ -875,7 +875,7 @@ void generateStringPrint(llvm::LLVMContext &context,
   }
 
   llvm::Value *cast = builder.CreatePointerCast(stringVar,
-                                                builder.getInt8PtrTy());
+                                                builder.getPtrTy());
   builder.CreateCall(printFunct, cast);
 }
 
@@ -919,7 +919,7 @@ void generateIntegerPrint(llvm::LLVMContext &context,
   }
 
   llvm::Value *cast = builder.CreateBitCast(stringVar,
-                                            builder.getInt8PtrTy());
+                                            builder.getPtrTy());
   builder.CreateCall(&printFunct, {&toPrint, cast});
 }
 
@@ -970,7 +970,7 @@ static llvm::BasicBlock *createFinallyBlock(llvm::LLVMContext &context,
                                          ourExceptionNotThrownState->getType(),
                                          ourExceptionNotThrownState);
 
-  llvm::PointerType *exceptionStorageType = builder.getInt8PtrTy();
+  llvm::PointerType *exceptionStorageType = builder.getPtrTy();
   *exceptionStorage = createEntryBlockAlloca(toAddTo,
                                              "exceptionStorage",
                                              exceptionStorageType,
@@ -1289,7 +1289,7 @@ static llvm::Function *createCatchWrappedInvokeFunction(
   typeInfoThrown = builder.CreateStructGEP(ourExceptionType, typeInfoThrown, 0);
 
   llvm::Value *typeInfoThrownType =
-      builder.CreateStructGEP(builder.getInt8PtrTy(), typeInfoThrown, 0);
+      builder.CreateStructGEP(builder.getPtrTy(), typeInfoThrown, 0);
 
   generateIntegerPrint(context,
                        module,
@@ -1614,7 +1614,7 @@ static void createStandardUtilityFunctions(unsigned numTypeInfos,
                                           TypeArray(builder.getInt32Ty()));
 
   llvm::Type *caughtResultFieldTypes[] = {
-    builder.getInt8PtrTy(),
+    builder.getPtrTy(),
     builder.getInt32Ty()
   };
 
@@ -1697,7 +1697,7 @@ static void createStandardUtilityFunctions(unsigned numTypeInfos,
 
   argTypes.clear();
   argTypes.push_back(builder.getInt32Ty());
-  argTypes.push_back(builder.getInt8PtrTy());
+  argTypes.push_back(builder.getPtrTy());
 
   argNames.clear();
 
@@ -1716,7 +1716,7 @@ static void createStandardUtilityFunctions(unsigned numTypeInfos,
 
   argTypes.clear();
   argTypes.push_back(builder.getInt64Ty());
-  argTypes.push_back(builder.getInt8PtrTy());
+  argTypes.push_back(builder.getPtrTy());
 
   argNames.clear();
 
@@ -1734,7 +1734,7 @@ static void createStandardUtilityFunctions(unsigned numTypeInfos,
   retType = builder.getVoidTy();
 
   argTypes.clear();
-  argTypes.push_back(builder.getInt8PtrTy());
+  argTypes.push_back(builder.getPtrTy());
 
   argNames.clear();
 
@@ -1770,7 +1770,7 @@ static void createStandardUtilityFunctions(unsigned numTypeInfos,
   retType = builder.getVoidTy();
 
   argTypes.clear();
-  argTypes.push_back(builder.getInt8PtrTy());
+  argTypes.push_back(builder.getPtrTy());
 
   argNames.clear();
 
@@ -1785,7 +1785,7 @@ static void createStandardUtilityFunctions(unsigned numTypeInfos,
 
   // createOurException
 
-  retType = builder.getInt8PtrTy();
+  retType = builder.getPtrTy();
 
   argTypes.clear();
   argTypes.push_back(builder.getInt32Ty());
@@ -1806,7 +1806,7 @@ static void createStandardUtilityFunctions(unsigned numTypeInfos,
   retType = builder.getInt32Ty();
 
   argTypes.clear();
-  argTypes.push_back(builder.getInt8PtrTy());
+  argTypes.push_back(builder.getPtrTy());
 
   argNames.clear();
 
@@ -1826,7 +1826,7 @@ static void createStandardUtilityFunctions(unsigned numTypeInfos,
   retType = builder.getInt32Ty();
 
   argTypes.clear();
-  argTypes.push_back(builder.getInt8PtrTy());
+  argTypes.push_back(builder.getPtrTy());
 
   argNames.clear();
 
@@ -1849,8 +1849,8 @@ static void createStandardUtilityFunctions(unsigned numTypeInfos,
   argTypes.push_back(builder.getInt32Ty());
   argTypes.push_back(builder.getInt32Ty());
   argTypes.push_back(builder.getInt64Ty());
-  argTypes.push_back(builder.getInt8PtrTy());
-  argTypes.push_back(builder.getInt8PtrTy());
+  argTypes.push_back(builder.getPtrTy());
+  argTypes.push_back(builder.getPtrTy());
 
   argNames.clear();
 

diff  --git a/llvm/unittests/Analysis/MemorySSATest.cpp b/llvm/unittests/Analysis/MemorySSATest.cpp
index 50a7d762a0215ed..4c157d5bedfc280 100644
--- a/llvm/unittests/Analysis/MemorySSATest.cpp
+++ b/llvm/unittests/Analysis/MemorySSATest.cpp
@@ -76,9 +76,8 @@ TEST_F(MemorySSATest, CreateALoad) {
   // We create a diamond where there is a store on one side, and then after
   // building MemorySSA, create a load after the merge point, and use it to test
   // updating by creating an access for the load.
-  F = Function::Create(
-      FunctionType::get(B.getVoidTy(), {B.getInt8PtrTy()}, false),
-      GlobalValue::ExternalLinkage, "F", &M);
+  F = Function::Create(FunctionType::get(B.getVoidTy(), {B.getPtrTy()}, false),
+                       GlobalValue::ExternalLinkage, "F", &M);
   BasicBlock *Entry(BasicBlock::Create(C, "", F));
   BasicBlock *Left(BasicBlock::Create(C, "", F));
   BasicBlock *Right(BasicBlock::Create(C, "", F));
@@ -114,9 +113,8 @@ TEST_F(MemorySSATest, CreateLoadsAndStoreUpdater) {
   // incrementally update it by inserting a store in the, entry, a load in the
   // merge point, then a store in the branch, another load in the merge point,
   // and then a store in the entry.
-  F = Function::Create(
-      FunctionType::get(B.getVoidTy(), {B.getInt8PtrTy()}, false),
-      GlobalValue::ExternalLinkage, "F", &M);
+  F = Function::Create(FunctionType::get(B.getVoidTy(), {B.getPtrTy()}, false),
+                       GlobalValue::ExternalLinkage, "F", &M);
   BasicBlock *Entry(BasicBlock::Create(C, "", F));
   BasicBlock *Left(BasicBlock::Create(C, "", F));
   BasicBlock *Right(BasicBlock::Create(C, "", F));
@@ -204,9 +202,8 @@ TEST_F(MemorySSATest, CreateALoadUpdater) {
   // We create a diamond, then build memoryssa with no memory accesses, and
   // incrementally update it by inserting a store in one of the branches, and a
   // load in the merge point
-  F = Function::Create(
-      FunctionType::get(B.getVoidTy(), {B.getInt8PtrTy()}, false),
-      GlobalValue::ExternalLinkage, "F", &M);
+  F = Function::Create(FunctionType::get(B.getVoidTy(), {B.getPtrTy()}, false),
+                       GlobalValue::ExternalLinkage, "F", &M);
   BasicBlock *Entry(BasicBlock::Create(C, "", F));
   BasicBlock *Left(BasicBlock::Create(C, "", F));
   BasicBlock *Right(BasicBlock::Create(C, "", F));
@@ -248,9 +245,8 @@ TEST_F(MemorySSATest, CreateALoadUpdater) {
 }
 
 TEST_F(MemorySSATest, SinkLoad) {
-  F = Function::Create(
-      FunctionType::get(B.getVoidTy(), {B.getInt8PtrTy()}, false),
-      GlobalValue::ExternalLinkage, "F", &M);
+  F = Function::Create(FunctionType::get(B.getVoidTy(), {B.getPtrTy()}, false),
+                       GlobalValue::ExternalLinkage, "F", &M);
   BasicBlock *Entry(BasicBlock::Create(C, "", F));
   BasicBlock *Left(BasicBlock::Create(C, "", F));
   BasicBlock *Right(BasicBlock::Create(C, "", F));
@@ -298,9 +294,8 @@ TEST_F(MemorySSATest, MoveAStore) {
   // a load at the end.  After building MemorySSA, we test updating by moving
   // the store from the side block to the entry block. This destroys the old
   // access.
-  F = Function::Create(
-      FunctionType::get(B.getVoidTy(), {B.getInt8PtrTy()}, false),
-      GlobalValue::ExternalLinkage, "F", &M);
+  F = Function::Create(FunctionType::get(B.getVoidTy(), {B.getPtrTy()}, false),
+                       GlobalValue::ExternalLinkage, "F", &M);
   BasicBlock *Entry(BasicBlock::Create(C, "", F));
   BasicBlock *Left(BasicBlock::Create(C, "", F));
   BasicBlock *Right(BasicBlock::Create(C, "", F));
@@ -334,9 +329,8 @@ TEST_F(MemorySSATest, MoveAStoreUpdater) {
   // a load at the end.  After building MemorySSA, we test updating by moving
   // the store from the side block to the entry block.  This destroys the old
   // access.
-  F = Function::Create(
-      FunctionType::get(B.getVoidTy(), {B.getInt8PtrTy()}, false),
-      GlobalValue::ExternalLinkage, "F", &M);
+  F = Function::Create(FunctionType::get(B.getVoidTy(), {B.getPtrTy()}, false),
+                       GlobalValue::ExternalLinkage, "F", &M);
   BasicBlock *Entry(BasicBlock::Create(C, "", F));
   BasicBlock *Left(BasicBlock::Create(C, "", F));
   BasicBlock *Right(BasicBlock::Create(C, "", F));
@@ -380,9 +374,8 @@ TEST_F(MemorySSATest, MoveAStoreUpdaterMove) {
   // a load at the end.  After building MemorySSA, we test updating by moving
   // the store from the side block to the entry block.  This does not destroy
   // the old access.
-  F = Function::Create(
-      FunctionType::get(B.getVoidTy(), {B.getInt8PtrTy()}, false),
-      GlobalValue::ExternalLinkage, "F", &M);
+  F = Function::Create(FunctionType::get(B.getVoidTy(), {B.getPtrTy()}, false),
+                       GlobalValue::ExternalLinkage, "F", &M);
   BasicBlock *Entry(BasicBlock::Create(C, "", F));
   BasicBlock *Left(BasicBlock::Create(C, "", F));
   BasicBlock *Right(BasicBlock::Create(C, "", F));
@@ -424,9 +417,8 @@ TEST_F(MemorySSATest, MoveAStoreAllAround) {
   // a load at the end.  After building MemorySSA, we test updating by moving
   // the store from the side block to the entry block, then to the other side
   // block, then to before the load.  This does not destroy the old access.
-  F = Function::Create(
-      FunctionType::get(B.getVoidTy(), {B.getInt8PtrTy()}, false),
-      GlobalValue::ExternalLinkage, "F", &M);
+  F = Function::Create(FunctionType::get(B.getVoidTy(), {B.getPtrTy()}, false),
+                       GlobalValue::ExternalLinkage, "F", &M);
   BasicBlock *Entry(BasicBlock::Create(C, "", F));
   BasicBlock *Left(BasicBlock::Create(C, "", F));
   BasicBlock *Right(BasicBlock::Create(C, "", F));
@@ -479,9 +471,8 @@ TEST_F(MemorySSATest, RemoveAPhi) {
   // We create a diamond where there is a store on one side, and then a load
   // after the merge point.  This enables us to test a bunch of 
diff erent
   // removal cases.
-  F = Function::Create(
-      FunctionType::get(B.getVoidTy(), {B.getInt8PtrTy()}, false),
-      GlobalValue::ExternalLinkage, "F", &M);
+  F = Function::Create(FunctionType::get(B.getVoidTy(), {B.getPtrTy()}, false),
+                       GlobalValue::ExternalLinkage, "F", &M);
   BasicBlock *Entry(BasicBlock::Create(C, "", F));
   BasicBlock *Left(BasicBlock::Create(C, "", F));
   BasicBlock *Right(BasicBlock::Create(C, "", F));
@@ -524,9 +515,8 @@ TEST_F(MemorySSATest, RemoveMemoryAccess) {
   // We create a diamond where there is a store on one side, and then a load
   // after the merge point.  This enables us to test a bunch of 
diff erent
   // removal cases.
-  F = Function::Create(
-      FunctionType::get(B.getVoidTy(), {B.getInt8PtrTy()}, false),
-      GlobalValue::ExternalLinkage, "F", &M);
+  F = Function::Create(FunctionType::get(B.getVoidTy(), {B.getPtrTy()}, false),
+                       GlobalValue::ExternalLinkage, "F", &M);
   BasicBlock *Entry(BasicBlock::Create(C, "", F));
   BasicBlock *Left(BasicBlock::Create(C, "", F));
   BasicBlock *Right(BasicBlock::Create(C, "", F));
@@ -885,9 +875,8 @@ TEST_F(MemorySSATest, Irreducible) {
 
   SmallVector<PHINode *, 8> Inserted;
   IRBuilder<> B(C);
-  F = Function::Create(
-      FunctionType::get(B.getVoidTy(), {B.getInt8PtrTy()}, false),
-      GlobalValue::ExternalLinkage, "F", &M);
+  F = Function::Create(FunctionType::get(B.getVoidTy(), {B.getPtrTy()}, false),
+                       GlobalValue::ExternalLinkage, "F", &M);
 
   // Make blocks
   BasicBlock *IfBB = BasicBlock::Create(C, "if", F);
@@ -924,9 +913,8 @@ TEST_F(MemorySSATest, MoveToBeforeLiveOnEntryInvalidatesCache) {
   // ...And be sure that MSSA's caching doesn't give us `1` for the clobber of
   // `2` after `1` is removed.
   IRBuilder<> B(C);
-  F = Function::Create(
-      FunctionType::get(B.getVoidTy(), {B.getInt8PtrTy()}, false),
-      GlobalValue::ExternalLinkage, "F", &M);
+  F = Function::Create(FunctionType::get(B.getVoidTy(), {B.getPtrTy()}, false),
+                       GlobalValue::ExternalLinkage, "F", &M);
 
   BasicBlock *Entry = BasicBlock::Create(C, "if", F);
   B.SetInsertPoint(Entry);
@@ -969,9 +957,8 @@ TEST_F(MemorySSATest, RemovingDefInvalidatesCache) {
   // And be sure that MSSA's caching handles the removal of def `1`
   // appropriately.
   IRBuilder<> B(C);
-  F = Function::Create(
-      FunctionType::get(B.getVoidTy(), {B.getInt8PtrTy()}, false),
-      GlobalValue::ExternalLinkage, "F", &M);
+  F = Function::Create(FunctionType::get(B.getVoidTy(), {B.getPtrTy()}, false),
+                       GlobalValue::ExternalLinkage, "F", &M);
 
   BasicBlock *Entry = BasicBlock::Create(C, "if", F);
   B.SetInsertPoint(Entry);
@@ -1045,10 +1032,9 @@ TEST_F(MemorySSATest, TestStoreMustAlias) {
 
 // Test May alias for optimized defs.
 TEST_F(MemorySSATest, TestStoreMayAlias) {
-  F = Function::Create(FunctionType::get(B.getVoidTy(),
-                                         {B.getInt8PtrTy(), B.getInt8PtrTy()},
-                                         false),
-                       GlobalValue::ExternalLinkage, "F", &M);
+  F = Function::Create(
+      FunctionType::get(B.getVoidTy(), {B.getPtrTy(), B.getPtrTy()}, false),
+      GlobalValue::ExternalLinkage, "F", &M);
   B.SetInsertPoint(BasicBlock::Create(C, "", F));
   Type *Int8 = Type::getInt8Ty(C);
   auto *ArgIt = F->arg_begin();
@@ -1117,9 +1103,8 @@ TEST_F(MemorySSATest, LifetimeMarkersAreClobbers) {
   // it.
 
   IRBuilder<> B(C);
-  F = Function::Create(
-      FunctionType::get(B.getVoidTy(), {B.getInt8PtrTy()}, false),
-      GlobalValue::ExternalLinkage, "F", &M);
+  F = Function::Create(FunctionType::get(B.getVoidTy(), {B.getPtrTy()}, false),
+                       GlobalValue::ExternalLinkage, "F", &M);
 
   // Make blocks
   BasicBlock *Entry = BasicBlock::Create(C, "entry", F);
@@ -1244,10 +1229,9 @@ TEST_F(MemorySSATest, DefOptimizationsAreInvalidatedOnMoving) {
 }
 
 TEST_F(MemorySSATest, TestOptimizedDefsAreProperUses) {
-  F = Function::Create(FunctionType::get(B.getVoidTy(),
-                                         {B.getInt8PtrTy(), B.getInt8PtrTy()},
-                                         false),
-                       GlobalValue::ExternalLinkage, "F", &M);
+  F = Function::Create(
+      FunctionType::get(B.getVoidTy(), {B.getPtrTy(), B.getPtrTy()}, false),
+      GlobalValue::ExternalLinkage, "F", &M);
   B.SetInsertPoint(BasicBlock::Create(C, "", F));
   Type *Int8 = Type::getInt8Ty(C);
   Value *AllocA = B.CreateAlloca(Int8, ConstantInt::get(Int8, 1), "A");
@@ -1322,9 +1306,8 @@ TEST_F(MemorySSATest, TestOptimizedDefsAreProperUses) {
 //  ; 4 = MemoryDef(3); optimized to 3, cannot optimize thorugh phi.
 //  Insert edge: entry -> exit, check mssa Update is correct.
 TEST_F(MemorySSATest, TestAddedEdgeToBlockWithPhiNotOpt) {
-  F = Function::Create(
-      FunctionType::get(B.getVoidTy(), {B.getInt8PtrTy()}, false),
-      GlobalValue::ExternalLinkage, "F", &M);
+  F = Function::Create(FunctionType::get(B.getVoidTy(), {B.getPtrTy()}, false),
+                       GlobalValue::ExternalLinkage, "F", &M);
   Argument *PointerArg = &*F->arg_begin();
   BasicBlock *Entry(BasicBlock::Create(C, "entry", F));
   BasicBlock *Header(BasicBlock::Create(C, "header", F));
@@ -1378,9 +1361,8 @@ TEST_F(MemorySSATest, TestAddedEdgeToBlockWithPhiNotOpt) {
 //  the optimized access.
 //  Insert edge: entry -> exit, check mssa Update is correct.
 TEST_F(MemorySSATest, TestAddedEdgeToBlockWithPhiOpt) {
-  F = Function::Create(
-      FunctionType::get(B.getVoidTy(), {B.getInt8PtrTy()}, false),
-      GlobalValue::ExternalLinkage, "F", &M);
+  F = Function::Create(FunctionType::get(B.getVoidTy(), {B.getPtrTy()}, false),
+                       GlobalValue::ExternalLinkage, "F", &M);
   Argument *PointerArg = &*F->arg_begin();
   Type *Int8 = Type::getInt8Ty(C);
   BasicBlock *Entry(BasicBlock::Create(C, "entry", F));
@@ -1450,9 +1432,8 @@ TEST_F(MemorySSATest, TestAddedEdgeToBlockWithPhiOpt) {
 // e:
 //  ; 2 = MemoryPhi({d, 4}, {f, 1})
 TEST_F(MemorySSATest, TestAddedEdgeToBlockWithNoPhiAddNewPhis) {
-  F = Function::Create(
-      FunctionType::get(B.getVoidTy(), {B.getInt8PtrTy()}, false),
-      GlobalValue::ExternalLinkage, "F", &M);
+  F = Function::Create(FunctionType::get(B.getVoidTy(), {B.getPtrTy()}, false),
+                       GlobalValue::ExternalLinkage, "F", &M);
   Argument *PointerArg = &*F->arg_begin();
   BasicBlock *Entry(BasicBlock::Create(C, "entry", F));
   BasicBlock *ABlock(BasicBlock::Create(C, "a", F));
@@ -1499,9 +1480,8 @@ TEST_F(MemorySSATest, TestAddedEdgeToBlockWithNoPhiAddNewPhis) {
 }
 
 TEST_F(MemorySSATest, TestCallClobber) {
-  F = Function::Create(
-      FunctionType::get(B.getVoidTy(), {B.getInt8PtrTy()}, false),
-      GlobalValue::ExternalLinkage, "F", &M);
+  F = Function::Create(FunctionType::get(B.getVoidTy(), {B.getPtrTy()}, false),
+                       GlobalValue::ExternalLinkage, "F", &M);
 
   Value *Pointer1 = &*F->arg_begin();
   BasicBlock *Entry(BasicBlock::Create(C, "", F));
@@ -1532,9 +1512,8 @@ TEST_F(MemorySSATest, TestCallClobber) {
 }
 
 TEST_F(MemorySSATest, TestLoadClobber) {
-  F = Function::Create(
-      FunctionType::get(B.getVoidTy(), {B.getInt8PtrTy()}, false),
-      GlobalValue::ExternalLinkage, "F", &M);
+  F = Function::Create(FunctionType::get(B.getVoidTy(), {B.getPtrTy()}, false),
+                       GlobalValue::ExternalLinkage, "F", &M);
 
   Value *Pointer1 = &*F->arg_begin();
   BasicBlock *Entry(BasicBlock::Create(C, "", F));

diff  --git a/llvm/unittests/Frontend/OpenMPIRBuilderTest.cpp b/llvm/unittests/Frontend/OpenMPIRBuilderTest.cpp
index 50759d7c61029d8..2876aa49da402e0 100644
--- a/llvm/unittests/Frontend/OpenMPIRBuilderTest.cpp
+++ b/llvm/unittests/Frontend/OpenMPIRBuilderTest.cpp
@@ -1899,7 +1899,7 @@ TEST_F(OpenMPIRBuilderTest, ApplySimdCustomAligned) {
   IRBuilder<> Builder(BB);
   const int AlignmentValue = 32;
   AllocaInst *Alloc1 =
-      Builder.CreateAlloca(Builder.getInt8PtrTy(), Builder.getInt64(1));
+      Builder.CreateAlloca(Builder.getPtrTy(), Builder.getInt64(1));
   LoadInst *Load1 = Builder.CreateLoad(Alloc1->getAllocatedType(), Alloc1);
   MapVector<Value *, Value *> AlignedVars;
   AlignedVars.insert({Load1, Builder.getInt64(AlignmentValue)});

diff  --git a/llvm/unittests/IR/InstructionsTest.cpp b/llvm/unittests/IR/InstructionsTest.cpp
index 20b8529b386324f..0ef3f66dbaee4e1 100644
--- a/llvm/unittests/IR/InstructionsTest.cpp
+++ b/llvm/unittests/IR/InstructionsTest.cpp
@@ -855,7 +855,7 @@ TEST_F(ModuleWithFunctionTest, DropPoisonGeneratingFlags) {
   }
 
   {
-    Value *GEPBase = Constant::getNullValue(B.getInt8PtrTy());
+    Value *GEPBase = Constant::getNullValue(B.getPtrTy());
     auto *GI = cast<GetElementPtrInst>(
         B.CreateInBoundsGEP(B.getInt8Ty(), GEPBase, Arg0));
     ASSERT_TRUE(GI->isInBounds());

diff  --git a/llvm/unittests/Transforms/Utils/FunctionComparatorTest.cpp b/llvm/unittests/Transforms/Utils/FunctionComparatorTest.cpp
index 42f9fb39c2d15d7..cd2b4e8046b2e44 100644
--- a/llvm/unittests/Transforms/Utils/FunctionComparatorTest.cpp
+++ b/llvm/unittests/Transforms/Utils/FunctionComparatorTest.cpp
@@ -26,7 +26,7 @@ struct TestFunction {
   TestFunction(LLVMContext &Ctx, Module &M, int addVal) {
     IRBuilder<> B(Ctx);
     T = B.getInt8Ty();
-    F = Function::Create(FunctionType::get(T, {B.getInt8PtrTy()}, false),
+    F = Function::Create(FunctionType::get(T, {B.getPtrTy()}, false),
                          GlobalValue::ExternalLinkage, "F", &M);
     BB = BasicBlock::Create(Ctx, "", F);
     B.SetInsertPoint(BB);


        


More information about the llvm-commits mailing list