[PATCH] D123662: [ArgPromotion] Rename variables according to the code style. NFC
Johannes Doerfert via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Apr 13 07:27:14 PDT 2022
jdoerfert accepted this revision.
jdoerfert added a comment.
This revision is now accepted and ready to land.
Thx! LG, one nit, see below.
================
Comment at: llvm/lib/Transforms/IPO/ArgumentPromotion.cpp:365
+ Align Alignment = commonAlignment(StructAlign, SL->getElementOffset(N));
new StoreInst(&*I2++, Idx, false, Alignment, InsertPt);
}
----------------
Make it `I` or something, not `N`. `N` is (for me) associated with the upper bound, also above.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D123662/new/
https://reviews.llvm.org/D123662
More information about the llvm-commits
mailing list