[PATCH] D26394: [AArch64] Fix incorrect CSEL node created

James Molloy via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 8 05:21:47 PST 2016


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

Hi Roger,

This looks like a sensible fix - thanks for doing this!

If I understand right, the reason your testcase doesn't have a FileCheck invocation is because the previous behaviour was a crash. Could you please mention this as a comment in the testcase?

With that fixed, LGTM.

James


https://reviews.llvm.org/D26394





More information about the llvm-commits mailing list