[PATCH] D71132: PostRA Machine Sink should take care of COPY defining register that is a sub-register by another COPY source operand

Valery Pykhtin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Dec 12 08:04:55 PST 2019


vpykhtin added a comment.

Almost LGTM. Do you need those liveins reorderings?



================
Comment at: llvm/test/CodeGen/AArch64/bisect-post-ra-machine-sink.mir:12
 # RUN-POSTRA: bb.1:
-# RUN-POSTRA: liveins: $w1, $w0
+# RUN-POSTRA: liveins: $w0, $w1
 # RUN-POSTRA: renamable $w19 = COPY killed $w0
----------------
is this really needed?


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

https://reviews.llvm.org/D71132





More information about the llvm-commits mailing list