diff --git a/clang/lib/CodeGen/CGBuilder.h b/clang/lib/CodeGen/CGBuilder.h --- a/clang/lib/CodeGen/CGBuilder.h +++ b/clang/lib/CodeGen/CGBuilder.h @@ -68,38 +68,34 @@ // take an alignment. llvm::LoadInst *CreateLoad(Address Addr, const llvm::Twine &Name = "") { return CreateAlignedLoad(Addr.getPointer(), - Addr.getAlignment().getQuantity(), - Name); + Addr.getAlignment().getAsAlign(), Name); } llvm::LoadInst *CreateLoad(Address Addr, const char *Name) { // This overload is required to prevent string literals from // ending up in the IsVolatile overload. return CreateAlignedLoad(Addr.getPointer(), - Addr.getAlignment().getQuantity(), - Name); + Addr.getAlignment().getAsAlign(), Name); } llvm::LoadInst *CreateLoad(Address Addr, bool IsVolatile, const llvm::Twine &Name = "") { - return CreateAlignedLoad(Addr.getPointer(), - Addr.getAlignment().getQuantity(), - IsVolatile, - Name); + return CreateAlignedLoad( + Addr.getPointer(), Addr.getAlignment().getAsAlign(), IsVolatile, Name); } using CGBuilderBaseTy::CreateAlignedLoad; llvm::LoadInst *CreateAlignedLoad(llvm::Value *Addr, CharUnits Align, const llvm::Twine &Name = "") { - return CreateAlignedLoad(Addr, Align.getQuantity(), Name); + return CreateAlignedLoad(Addr, Align.getAsAlign(), Name); } llvm::LoadInst *CreateAlignedLoad(llvm::Value *Addr, CharUnits Align, const char *Name) { - return CreateAlignedLoad(Addr, Align.getQuantity(), Name); + return CreateAlignedLoad(Addr, Align.getAsAlign(), Name); } llvm::LoadInst *CreateAlignedLoad(llvm::Type *Ty, llvm::Value *Addr, CharUnits Align, const llvm::Twine &Name = "") { assert(Addr->getType()->getPointerElementType() == Ty); - return CreateAlignedLoad(Addr, Align.getQuantity(), Name); + return CreateAlignedLoad(Addr, Align.getAsAlign(), Name); } // Note that we intentionally hide the CreateStore APIs that don't diff --git a/clang/lib/CodeGen/CGCXX.cpp b/clang/lib/CodeGen/CGCXX.cpp --- a/clang/lib/CodeGen/CGCXX.cpp +++ b/clang/lib/CodeGen/CGCXX.cpp @@ -263,8 +263,8 @@ AddressPoint.AddressPointIndex; llvm::Value *VFuncPtr = CGF.Builder.CreateConstInBoundsGEP1_64(VTable, VTableIndex, "vfnkxt"); - llvm::Value *VFunc = - CGF.Builder.CreateAlignedLoad(VFuncPtr, CGF.PointerAlignInBytes); + llvm::Value *VFunc = CGF.Builder.CreateAlignedLoad( + VFuncPtr, llvm::Align(CGF.PointerAlignInBytes)); CGCallee Callee(GD, VFunc); return Callee; } diff --git a/clang/lib/CodeGen/CodeGenFunction.h b/clang/lib/CodeGen/CodeGenFunction.h --- a/clang/lib/CodeGen/CodeGenFunction.h +++ b/clang/lib/CodeGen/CodeGenFunction.h @@ -4414,7 +4414,7 @@ // Otherwise, it should be an alloca instruction, as set up in save(). auto alloca = cast(value.getPointer()); - return CGF.Builder.CreateAlignedLoad(alloca, alloca->getAlignment()); + return CGF.Builder.CreateAlignedLoad(alloca, alloca->getAlign()); } } // end namespace CodeGen diff --git a/llvm/include/llvm/IR/IRBuilder.h b/llvm/include/llvm/IR/IRBuilder.h --- a/llvm/include/llvm/IR/IRBuilder.h +++ b/llvm/include/llvm/IR/IRBuilder.h @@ -1819,19 +1819,27 @@ } // Deprecated [opaque pointer types] - LoadInst *CreateAlignedLoad(Value *Ptr, unsigned Align, const char *Name) { + LLVM_ATTRIBUTE_DEPRECATED(LoadInst *CreateAlignedLoad(Value *Ptr, + unsigned Align, + const char *Name), + "Use the version that takes MaybeAlign instead") { return CreateAlignedLoad(Ptr->getType()->getPointerElementType(), Ptr, MaybeAlign(Align), Name); } // Deprecated [opaque pointer types] - LoadInst *CreateAlignedLoad(Value *Ptr, unsigned Align, - const Twine &Name = "") { + LLVM_ATTRIBUTE_DEPRECATED(LoadInst *CreateAlignedLoad(Value *Ptr, + unsigned Align, + const Twine &Name = ""), + "Use the version that takes MaybeAlign instead") { return CreateAlignedLoad(Ptr->getType()->getPointerElementType(), Ptr, MaybeAlign(Align), Name); } // Deprecated [opaque pointer types] - LoadInst *CreateAlignedLoad(Value *Ptr, unsigned Align, bool isVolatile, - const Twine &Name = "") { + LLVM_ATTRIBUTE_DEPRECATED(LoadInst *CreateAlignedLoad(Value *Ptr, + unsigned Align, + bool isVolatile, + const Twine &Name = ""), + "Use the version that takes MaybeAlign instead") { return CreateAlignedLoad(Ptr->getType()->getPointerElementType(), Ptr, MaybeAlign(Align), isVolatile, Name); } diff --git a/llvm/lib/Transforms/Scalar/LowerMatrixIntrinsics.cpp b/llvm/lib/Transforms/Scalar/LowerMatrixIntrinsics.cpp --- a/llvm/lib/Transforms/Scalar/LowerMatrixIntrinsics.cpp +++ b/llvm/lib/Transforms/Scalar/LowerMatrixIntrinsics.cpp @@ -517,8 +517,8 @@ LoadInst *createColumnLoad(Value *ColumnPtr, Type *EltType, IRBuilder<> Builder) { - unsigned Align = DL.getABITypeAlignment(EltType); - return Builder.CreateAlignedLoad(ColumnPtr, Align, "col.load"); + return Builder.CreateAlignedLoad( + ColumnPtr, Align(DL.getABITypeAlignment(EltType)), "col.load"); } StoreInst *createColumnStore(Value *ColumnValue, Value *ColumnPtr, diff --git a/polly/lib/CodeGen/BlockGenerators.cpp b/polly/lib/CodeGen/BlockGenerators.cpp --- a/polly/lib/CodeGen/BlockGenerators.cpp +++ b/polly/lib/CodeGen/BlockGenerators.cpp @@ -316,8 +316,8 @@ Value *NewPointer = generateLocationAccessed(Stmt, Load, BBMap, LTS, NewAccesses); - Value *ScalarLoad = Builder.CreateAlignedLoad( - NewPointer, Load->getAlignment(), Load->getName() + "_p_scalar_"); + Value *ScalarLoad = Builder.CreateAlignedLoad(NewPointer, Load->getAlign(), + Load->getName() + "_p_scalar_"); if (PollyDebugPrinting) RuntimeDebugBuilder::createCPUPrinter(Builder, "Load from ", NewPointer,