[PATCH] D59633: [PowerPC] Look through copies for compare elimination

[Stefan Pintilie via Phabricator via llvm-commits]

stefanp accepted this revision.
stefanp added a comment.

Have a couple of minor nits about the test case but that is all.
LGTM.



================
Comment at: test/CodeGen/PowerPC/eliminate-compare-of-copy.ll:7
+; CHECK: or. {{r[0-9]+}}, r4, r3
+; CHECK-NOT: cmplwi
+entry:
----------------
I usually add a `CHECK: blr` at the end so that I know where the end of the function is. In a way it is just like adding the CHECK-LABEL at the start.



================
Comment at: test/CodeGen/PowerPC/eliminate-compare-of-copy.ll:14
+if.then:                                          ; preds = %entry
+  %call = tail call signext i32 bitcast (i32 (...)* @f1 to i32 ()*)() #2
+  br label %return
----------------
nit:
Cleanup: I don't think that you need the `#0`, `#1` or `#2`. 


Repository:
  rL LLVM

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

https://reviews.llvm.org/D59633