[PATCH] D114611: [AVR] Expand STDWSPQRr & STDSPQRr, approach #2

Patryk Wychowaniec via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 18 02:50:26 PST 2022


Patryk27 added a comment.

I've just found a small bug with our current approach: if we generate `STDWPtrQRr` from within `AVRExpandPseudo`, then we don't have a chance to run `AVRRelaxMem` on that newly-created instruction (so, bottom line, functions with larger stacks trigger `assert(Imm <= 62 && "Offset is out of range");`).

Maybe we should have a separate pass for `STDWPtrQRr`?


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

https://reviews.llvm.org/D114611



More information about the llvm-commits mailing list