[llvm] Use more explicit cast methods (NFC)

Instead of ConstantExpr::getCast() with a fixed opcode, use the
corresponding getXYZ methods instead. For the one place creating
a pointer bitcast drop it entirely, as this is redundant with
opaque pointers.

Author: nikic

llvm/lib/Analysis/ConstantFolding.cpp

@@ -588,7 +588,7 @@ Constant *FoldReinterpretLoadFromConst(Constant *C, Type *LoadTy,
         if (DL.isNonIntegralPointerType(LoadTy->getScalarType()))
           // Be careful not to replace a load of an addrspace value with an inttoptr here
           return nullptr;
-        Res = ConstantExpr::getCast(Instruction::IntToPtr, Res, LoadTy);
+        Res = ConstantExpr::getIntToPtr(Res, LoadTy);
       }
       return Res;
     }

llvm/lib/CodeGen/CodeGenPrepare.cpp

@@ -6305,7 +6305,7 @@ bool CodeGenPrepare::optimizePhiType(
   // correct type.
   ValueToValueMap ValMap;
   for (ConstantData *C : Constants)
-    ValMap[C] = ConstantExpr::getCast(Instruction::BitCast, C, ConvertTy);
+    ValMap[C] = ConstantExpr::getBitCast(C, ConvertTy);
   for (Instruction *D : Defs) {
     if (isa<BitCastInst>(D)) {
       ValMap[D] = D->getOperand(0);

llvm/lib/ExecutionEngine/Orc/IndirectionUtils.cpp

@@ -244,8 +244,7 @@ Constant* createIRTypedAddress(FunctionType &FT, ExecutorAddr Addr) {
   Constant *AddrIntVal =
     ConstantInt::get(Type::getInt64Ty(FT.getContext()), Addr.getValue());
   Constant *AddrPtrVal =
-    ConstantExpr::getCast(Instruction::IntToPtr, AddrIntVal,
-                          PointerType::get(&FT, 0));
+    ConstantExpr::getIntToPtr(AddrIntVal, PointerType::get(&FT, 0));
   return AddrPtrVal;
 }
 

llvm/unittests/Analysis/SparsePropagation.cpp

@@ -488,7 +488,7 @@ TEST_F(SparsePropagationTest, ComputeInstructionState) {
 ///
 /// declare internal void @g()
 ///
-/// define internal void @f() personality i8* bitcast (void ()* @p to i8*) {
+/// define internal void @f() personality ptr @p {
 /// entry:
 ///   invoke void @g()
 ///           to label %exit unwind label %catch.pad
@@ -513,9 +513,7 @@ TEST_F(SparsePropagationTest, ExceptionalTerminatorInsts) {
                                  GlobalValue::InternalLinkage, "g", &M);
   Function *F = Function::Create(FunctionType::get(Builder.getVoidTy(), false),
                                  GlobalValue::InternalLinkage, "f", &M);
-  Constant *C =
-      ConstantExpr::getCast(Instruction::BitCast, P, Builder.getInt8PtrTy());
-  F->setPersonalityFn(C);
+  F->setPersonalityFn(P);
   BasicBlock *Entry = BasicBlock::Create(Context, "entry", F);
   BasicBlock *Pad = BasicBlock::Create(Context, "catch.pad", F);
   BasicBlock *Body = BasicBlock::Create(Context, "catch.body", F);

llvm/unittests/Linker/LinkModulesTest.cpp

@@ -51,8 +51,7 @@ class LinkModuleTest : public testing::Test {
     Init.push_back(SwitchCase2BA);
 
     ConstantInt *One = ConstantInt::get(Type::getInt32Ty(Ctx), 1);
-    Constant *OnePtr = ConstantExpr::getCast(Instruction::IntToPtr, One,
-                                             Type::getInt8PtrTy(Ctx));
+    Constant *OnePtr = ConstantExpr::getIntToPtr(One, Type::getInt8PtrTy(Ctx));
     Init.push_back(OnePtr);
 
     GV->setInitializer(ConstantArray::get(AT, Init));