[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