[PATCH] D53236: [SelectionDAG] swap select_cc operands to enable folding

Sanjay Patel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 15 10:07:58 PDT 2018


spatel added a comment.

Is the motivating case integer or FP?
I'm asking because we have a canonicalization for integer cmp+sel for the IR in these tests, but we're missing the corresponding FP transform. 
If we add the FP canonicalization in IR, would there still be a need for this backend patch? Ie, is something generating this select code in the DAG itself?


https://reviews.llvm.org/D53236





More information about the llvm-commits mailing list