[PATCH] D17475: MachineCopyPropagation: Catch copies of the form A<-B; A<-B

[Quentin Colombet via llvm-commits]

qcolombet accepted this revision.
qcolombet added a comment.
This revision is now accepted and ready to land.

LGTM plus maybe a test for reserved registers (i.e., don’t do anything for those).

Q.


================
Comment at: test/CodeGen/AMDGPU/llvm.amdgcn.rsq.clamp.ll:34
@@ -33,4 +33,3 @@
 ; VI: s_mov_b32 s[[HIGH2:[0-9+]]], 0xffefffff
-; VI: s_mov_b32 s[[LOW2:[0-9+]]], s[[ALLBITS]]
-; VI: v_max_f64 v[0:1], v[0:1], s{{\[}}[[LOW2]]:[[HIGH2]]]
+; VI: v_max_f64 v[0:1], v[0:1], s{{\[}}[[LOW]]:[[HIGH2]]]
 define void @rsq_clamp_f64(double addrspace(1)* %out, double %src) #0 {
----------------
I guess this is the rebase messing around.

================
Comment at: test/CodeGen/X86/machine-copy-prop.mir:184
@@ +183,2 @@
+    NOOP implicit %rax, implicit %rbp
+...
----------------
Could you add a test with Reserved register for both cases source and destination?


Repository:
  rL LLVM

http://reviews.llvm.org/D17475