RFR: 8324833: Signed integer overflows in ABS [v2]

Aleksey Shipilev shade at openjdk.org
Fri Feb 2 06:23:02 UTC 2024


On Wed, 31 Jan 2024 15:55:25 GMT, Aleksey Shipilev <shade at openjdk.org> wrote:

>> 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!)
>>  - [x] Linux x86_64 fastdebug, `all`
>
> Aleksey Shipilev has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Unnecessary comment

Yes, I think we care. I would try a build with `Wsigned-conversion` and look where we should sprinkle more `checked_cast`-s.

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

PR Comment: https://git.openjdk.org/jdk/pull/17617#issuecomment-1922955956


More information about the graal-dev mailing list