[PATCH] D29639: [SelectionDAG] Add a signed integer absolute ISD node

Krzysztof Parzyszek via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 14 07:31:15 PDT 2017


kparzysz added a comment.

In https://reviews.llvm.org/D29639#678538, @RKSimon wrote:

> Thanks Michael - does anyone from a non-X86 target have any comments?


Hexagon has integer ABS for 32- and 64-bit integers, and for both of them abs(min_int) == min_int, so this works for us.  FWIW, we also have a saturating abs for 32-bit integers.


Repository:
  rL LLVM

https://reviews.llvm.org/D29639





More information about the llvm-commits mailing list