[PATCH] D63060: [InstCombine] usub.sat(a, b) + b => umax(a, b) (PR42178)

Roman Lebedev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 24 06:57:17 PDT 2019


lebedev.ri added a comment.

In D63060#1555484 <https://reviews.llvm.org/D63060#1555484>, @xbolva00 wrote:

> I would like to ask if the increase of uses due to new transformation is generally banned in InstCombine (any pattern which we would like to fold to min/max pattern) or not? If banned, what is acceptable solution for such cases ?


*uses* or *instructions*? Only instruction count should not increase.
use-count is unspecified, unless of course it's illegal as per `undef` semantics.

> @spatel @efriedma




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