[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