[PATCH] D59506: [ValueTracking][InstSimplify] Support min/max selects in computeConstantRange()

Nikita Popov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Apr 7 10:36:56 PDT 2019


nikic added a comment.

In D59506#1457375 <https://reviews.llvm.org/D59506#1457375>, @arsenm wrote:

> LGTM, although if -O0 works that would be preferable to introducing a new pass control flag


Gave this a try, with `-O0` I end up with both v_max/v_min operands being non-immediate.


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D59506/new/

https://reviews.llvm.org/D59506





More information about the llvm-commits mailing list