[clang] [llvm] [llvm] Remove GlobalVariable::getAlignment (NFC) (PR #213300)
Matt Arsenault via cfe-commits
cfe-commits at lists.llvm.org
Fri Jul 31 08:56:15 PDT 2026
================
@@ -587,9 +587,9 @@ static void replaceManagedVar(llvm::GlobalVariable *Var,
}
if (auto *I = dyn_cast<llvm::Instruction>(U)) {
llvm::Value *OldV = Var;
- llvm::Instruction *NewV = new llvm::LoadInst(
- Var->getType(), ManagedVar, "ld.managed", false,
- llvm::Align(Var->getAlignment()), I->getIterator());
+ llvm::Instruction *NewV =
+ new llvm::LoadInst(Var->getType(), ManagedVar, "ld.managed", false,
+ Var->getAlign().valueOrOne(), I->getIterator());
----------------
arsenm wrote:
Can we get rid of MaybeAlign? Adding that was a big mistake, and the default alignment should always be 1
https://github.com/llvm/llvm-project/pull/213300
More information about the cfe-commits
mailing list