[PATCH] D34743: [AArch64] AArch64CondBrTuningPass generates wrong branch instructions

Chad Rosier via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 28 11:01:30 PDT 2017


mcrosier added a comment.

Thanks for the fix, Alexandros!

I'll improve the test cases shortly, Geoff.



================
Comment at: lib/Target/AArch64/AArch64CondBrTuning.cpp:25
 /// 3) sub w8, w0, w1       -> subs w8, w0, w1   ; w8 has multiple uses.
 ///    tbz w8, #31, .LBB6_2 -> b.ge .LBB6_2
 ///
----------------
gberry wrote:
> This comment should be fixed.
I'll update this shortly.


Repository:
  rL LLVM

https://reviews.llvm.org/D34743





More information about the llvm-commits mailing list