[PATCH] D107692: [DAGCombine] Prevent the transform of combine for multi-use operand
Roman Lebedev via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Aug 16 05:39:19 PDT 2021
lebedev.ri requested changes to this revision.
lebedev.ri added a comment.
This revision now requires changes to proceed.
Please,
1. either add a comment explaining why the one-use check needed
2. or rewrite the transform the right way, so that the lack of one-use check shows up as extra instruction bloat and not a miscompilation
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D107692/new/
https://reviews.llvm.org/D107692
More information about the llvm-commits
mailing list