[PATCH] D53478: [DAGCombine] SimplifyNodeWithTwoResults - ensure same legalization for LO/HI operands (PR21207)
Nirav Dave via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Oct 22 08:01:35 PDT 2018
niravd added a comment.
I don't think that need to avoid custom operations post-legalization. The documentation doesn't seem to address this, but as I understand it, Custom is Legal but with non-standard lowering and we just happen to lower custom node in legalize as an easy way to sure we've legalized the DAG.
Can we solve the inconsistency the other way and allow custom in the other cases (L3568, L3584, L3594)?
Repository:
rL LLVM
https://reviews.llvm.org/D53478
More information about the llvm-commits
mailing list