[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