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

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 23 09:29:18 PDT 2019


fhahn added inline comments.


================
Comment at: llvm/include/llvm/Transforms/Scalar/GVNExpression.h:326
   LoadInst *Load;
-  unsigned Alignment;
+  llvm::MaybeAlign Alignment;
 
----------------
This is wrapped in `namespace llvm {`. Is MaybeAlign ambiguous in this context?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D67922





More information about the llvm-commits mailing list