[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