[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