[PATCH] D140569: [AVR] Custom lower 32-bit shift instructions
Ben Shi via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Sun Dec 25 19:51:28 PST 2022
benshi001 added inline comments.
================
Comment at: llvm/lib/Target/AVR/AVRISelLowering.cpp:1847
+ // Shift one to the left.
+ for (size_t i = 0; i < Regs.size(); i++) {
+ size_t Idx = Regs.size() - i - 1;
----------------
aykevl wrote:
> benshi001 wrote:
> > Would it be better to be
> > ```
> > for (ssize_t i = Regs.size() - 1; i >= 0; i--) {
> > ...
> > if (i == Regs.size() - 1) {
> > ...
> > } else {
> > ...
> > }
> > }
> > ```
> Fixed locally.
I saw you use captical loop variable `I` in your other patch, https://reviews.llvm.org/D140570, maybe it would be better to also use `I` instead of `i` here.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D140569/new/
https://reviews.llvm.org/D140569
More information about the llvm-commits
mailing list