[PATCH] D30364: AArch64 : Add FastCSEL feature.

Junmo Park via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 6 15:46:08 PST 2017


flyingforyou added a comment.

@evandro 
This is code review about the case "Some targets using CSEL is more prefer than CSINV, CSINC.".

And there were good discussions which way is better & correct for handling the issue. As you said, there is no specific target for this feature,now. 
I just wanted to get a confirm from other members, How to handle when we want to do this.

@kristof.beyls 
Thanks for comments. Your suggestion looks better.


https://reviews.llvm.org/D30364





More information about the llvm-commits mailing list