[PATCH] D77121: [Alignment][NFC] Transitionning more getMachineMemOperand call sites

Clement Courbet via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 31 01:37:08 PDT 2020


courbet accepted this revision.
courbet added inline comments.
This revision is now accepted and ready to land.
Herald added a subscriber: wuzish.


================
Comment at: llvm/lib/Target/PowerPC/PPCISelLowering.h:994
       bool IsInvariant = false;
-      unsigned Alignment = 0;
+      Align Alignment;
       AAMDNodes AAInfo;
----------------
This is a potential functional change, so I'll mention what I checked for posterity:
There are three instantiations of this class. In all cases, `Alignment` is never read before being written to.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D77121





More information about the llvm-commits mailing list