[PATCH] D67922: [Alignment][NFC] MaybeAlign in GVNExpression

[Florian Hahn via Phabricator via llvm-commits]

fhahn added inline comments.


================
Comment at: llvm/lib/Transforms/Scalar/NewGVN.cpp:1335
   if (LI)
-    E->setAlignment(LI->getAlignment());
+    E->setAlignment(llvm::MaybeAlign(LI->getAlignment()));
 
----------------
`llvm::` should not be needed here either. I think in almost all contexts in `llvm/`, it should not be needed and probably should be removed in the patches already submitted as well.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D67922