[PATCH] D77454: LoadInst should store Align, not MaybeAlign.

Guillaume Chatelet via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed May 13 05:54:34 PDT 2020


gchatelet added a comment.

Is this patch still supposed to land? If so can you format and rebase?



================
Comment at: llvm/lib/Transforms/Scalar/SROA.cpp:1278
   SomeLoad->getAAMetadata(AATags);
-  const MaybeAlign Align = MaybeAlign(SomeLoad->getAlignment());
+  llvm::Align Align = SomeLoad->getAlign();
 
----------------
rename `Align` to `Alignment` and get rid of `llvm::`


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D77454/new/

https://reviews.llvm.org/D77454





More information about the llvm-commits mailing list