RFR: 8324833: Signed integer overflows in ABS

Aleksey Shipilev shade at openjdk.org
Wed Jan 31 08:26:10 UTC 2024


See the details in the bug. I think current `ABS` implementation is beyond repair, and we should just switch to `uabs`.

Additional testing:
 - [x] Linux x86_64 fastdebug, `all` with `-ftrapv` (now fully passes!)
 - [ ] Linux x86_64 fastdebug, `all`

-------------

Commit messages:
 - Also handle the -stride_con with proper uabs
 - Another attempt at JVMCI fix
 - Work
 - Cleaner version
 - Merge branch 'master' into JDK-8324833-uabs
 - Rewrite JVMCI usage to avoid the messy overload
 - Update

Changes: https://git.openjdk.org/jdk/pull/17617/files
 Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=17617&range=00
  Issue: https://bugs.openjdk.org/browse/JDK-8324833
  Stats: 23 lines in 8 files changed: 2 ins; 5 del; 16 mod
  Patch: https://git.openjdk.org/jdk/pull/17617.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/17617/head:pull/17617

PR: https://git.openjdk.org/jdk/pull/17617


More information about the graal-dev mailing list