Re: RFR: 8316645: RISC-V: Remove dependency on libatomic by adding cmpxchg 1b

2023-10-01 Thread Robbin Ehn
On Fri, 29 Sep 2023 16:27:48 GMT, Hamlin Li  wrote:

>> Hi all, please consider.
>> 
>> latomic is used for non native atomic operation which causes problems with 
>> extra dependency.
>> This have been fixed in recent gcc, so latomic is no longer needed.
>> 
>> Added new gtest, passes t1 on vf2/qemu.
>
> Nice!

@Hamlin-Li thanks!

-

PR Comment: https://git.openjdk.org/jdk/pull/15919#issuecomment-1742131713


Integrated: 8316645: RISC-V: Remove dependency on libatomic by adding cmpxchg 1b

2023-10-01 Thread Robbin Ehn
On Tue, 26 Sep 2023 12:04:49 GMT, Robbin Ehn  wrote:

> Hi all, please consider.
> 
> latomic is used for non native atomic operation which causes problems with 
> extra dependency.
> This have been fixed in recent gcc, so latomic is no longer needed.
> 
> Added new gtest, passes t1 on vf2/qemu.

This pull request has now been integrated.

Changeset: fb055e7e
Author:Robbin Ehn 
URL:   
https://git.openjdk.org/jdk/commit/fb055e7e5300958b2a6a290aa6783e8ede929d9a
Stats: 124 lines in 3 files changed: 118 ins; 6 del; 0 mod

8316645: RISC-V: Remove dependency on libatomic by adding cmpxchg 1b

Reviewed-by: ihse, fyang, luhenry, mli

-

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


Re: RFR: 8314488: Compile the JDK as C++17

2023-10-01 Thread Martin Doerr
On Mon, 24 Jul 2023 01:41:16 GMT, Julian Waters  wrote:

> Implementation of [JEP draft: Compile the JDK as 
> C++17](https://bugs.openjdk.org/browse/JDK-8310260)

We're not changing it for existing releases. I don't think non-LTS releases 
play a significant role regarding such compatibility. Next LTS is supposed to 
be JDK 25 (2025-09-16, https://www.java.com/releases/).

-

PR Comment: https://git.openjdk.org/jdk/pull/14988#issuecomment-1742099242


Re: RFR: 8314488: Compile the JDK as C++17

2023-10-01 Thread Michael Felt
On Mon, 24 Jul 2023 01:41:16 GMT, Julian Waters  wrote:

> Implementation of [JEP draft: Compile the JDK as 
> C++17](https://bugs.openjdk.org/browse/JDK-8310260)

Requiring xlc17 aka openxl means any AIX rte requirements change as well. 
Binary compatibility, rte , etc.

-

PR Comment: https://git.openjdk.org/jdk/pull/14988#issuecomment-1741985638