[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