[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