On Thu, 3 Sep 2026 04:59:32 GMT, Quan Anh Mai <[email protected]> wrote:

>> Hi,
>> 
>> This PR implements addition and subtraction of 128-bit integers in C2. This 
>> may have numerous applications:
>> 
>> - It helps intrinsification of operations on 128-bit integers when we have 
>> one.
>> - It allows computation that may be incorrect at `long` due to overflow, 
>> such as loop predication of long range checks.
>> 
>> By implementing this separately, we don't need to worry about finalizing a 
>> public API, and we can have a better testing coverage.
>> 
>> Testing:
>> 
>> - [x] tier1-4,hs-comp-stress
>> 
>> Please take a look and leave your review, thanks a lot.
>> 
>> ---------
>> - [x] I confirm that I make this contribution in accordance with the 
>> [OpenJDK Interim AI Policy](https://openjdk.org/legal/ai).
>
> Quan Anh Mai has updated the pull request with a new target base due to a 
> merge or a rebase. The pull request now contains eight commits:
> 
>  - Merge branch 'master' into int128
>  - Use TypeTuple::LONG_PAIR
>  - Merge branch 'master' into int128
>  - C++17
>  - Merge branch 'master' into int128
>  - Merge branch 'master' into int128
>  - whitespace
>  - Implement Int128T addsub

I can't find the reason for the "T" in "128T" anywhere.  Is it a clever play on 
C++ "int32_t" and "int64_t"?  If so I'm not sure it's useful here.

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

PR Comment: https://git.openjdk.org/jdk/pull/31008#issuecomment-5521588586

Reply via email to