RFR: 8344304: [s390x] ubsan: negation of -2147483648 cannot be represented in type 'int' [v5]

Dean Long dlong at openjdk.org
Tue Dec 3 11:58:39 UTC 2024


On Tue, 3 Dec 2024 11:34:19 GMT, Amit Kumar <amitkumar at openjdk.org> wrote:

>> fixes the issue reported by ubsan.
>
> Amit Kumar has updated the pull request incrementally with one additional commit since the last revision:
> 
>   make similar to add2reg

src/hotspot/cpu/s390/macroAssembler_s390.hpp line 160:

> 158:   // Generic operation r1 := r2 + imm.
> 159:   void add2reg   (Register r1, int64_t imm, Register r2 = noreg);
> 160:   void add2reg_32(Register r1, int64_t imm, Register r2 = noreg);

I don't understand the difference between these two.  For both, imm must be a simm_32.  I don't think we need add2reg_32.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/22456#discussion_r1867585614


More information about the hotspot-compiler-dev mailing list