================
@@ -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
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to