[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