[PATCH] D63060: [InstCombine] usub.sat(a, b) + b => umax(a, b) (PR42178)
Dávid Bolvanský via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Jun 24 07:00:43 PDT 2019
xbolva00 added a comment.
Thanks for answer.
This comment:
“The transform is not correct, because it increases the number of uses of A. If A is undef, it may evaluate to a different value for each use :(“
Why this is issue ? since we can check if A is undef
Repository:
rL LLVM
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D63060/new/
https://reviews.llvm.org/D63060
More information about the llvm-commits
mailing list