[PATCH] D128588: [AVR] Fix expanding MOVW for overlapping registers

Patryk Wychowaniec via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Jun 26 01:11:03 PDT 2022


Patryk27 marked 2 inline comments as done.
Patryk27 added inline comments.


================
Comment at: llvm/test/CodeGen/AVR/pseudo/COPY.mir:31
+    $r23r22 = COPY $r25r24
+    RCALLk @foo, implicit $sp, implicit killed $r24r23
+...
----------------
benshi001 wrote:
> Are these two `implicit killed` in the two RCALLk necessary ?
Without any instruction that actually uses the result of that `COPY`, LLVM seems to generate no code whatsoever - I've thought about `RCALLk` as a simple way of telling LLVM "trust me, that register is read later".

Those don't have to be `killed` though, so to simplify the test, I've just changed them to `RCALLk @foo, implicit $r24r23`.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D128588



More information about the llvm-commits mailing list