[PATCH] D87188: [InstCombine] Canonicalize SPF to abs intrinc (WIP)

Nikita Popov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Sep 6 11:58:08 PDT 2020


nikic added a comment.

I've looked at some of the diffs on test suite. The main change I'm seeing is that loops containing abs now get unrolled more. I'm not sure whether that's expected/desired or not.

I've also spotted two possible folds (these aren't regressions though, they aren't recognized in select form either):

- abs(x, false) < 0 => x == INT_MIN
- https://alive2.llvm.org/ce/z/Mu93Wx


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

https://reviews.llvm.org/D87188



More information about the llvm-commits mailing list