Re: RFR: 8363910: Avoid tuning for Power10 CPUs on Linux ppc64le when gcc < 10 is used

2025-07-25 Thread Matthias Baesken
On Thu, 24 Jul 2025 16:30:08 GMT, Lutz Schmidt wrote: > Just for my understanding: If `-mtune=power10` is already in the arguments, > nothing is changed/added. In any other case (e.g. `-mtune=power11`), the > string `-mtune=power8` is added. Is that intended? We do not set '-mtune=power11' ,

Integrated: 8363910: Avoid tuning for Power10 CPUs on Linux ppc64le when gcc < 10 is used

2025-07-25 Thread Matthias Baesken
On Wed, 23 Jul 2025 12:23:04 GMT, Matthias Baesken wrote: > Older gcc version (8, 9) do not support the -mtune=power10 flag. We should > set this only when recent gcc versions are used. This pull request has now been integrated. Changeset: 41c94eed Author:Matthias Baesken URL: http

Re: RFR: 8363910: Avoid tuning for Power10 CPUs on Linux ppc64le when gcc < 10 is used

2025-07-25 Thread Matthias Baesken
On Wed, 23 Jul 2025 12:23:04 GMT, Matthias Baesken wrote: > Older gcc version (8, 9) do not support the -mtune=power10 flag. We should > set this only when recent gcc versions are used. Thanks for the review ! - PR Comment: https://git.openjdk.org/jdk/pull/26442#issuecomment-31174

Re: RFR: 8363063: AArch64: [VectorAPI] sve vector math operations are not supported after JDK-8353217

2025-07-25 Thread Andrew Haley
On Thu, 24 Jul 2025 16:02:41 GMT, Fei Gao wrote: > This patch fixes a typo introduced in > [JDK-8353217](https://github.com/openjdk/jdk/commit/130b0cdaa6604da47a893e5425547acf3d5253f4), > which incorrectly disabled SVE vector math symbols. As a result, some vector > math test cases such as `jd

Re: RFR: 8363910: Avoid tuning for Power10 CPUs on Linux ppc64le when gcc < 10 is used

2025-07-25 Thread Thomas Stuefe
On Wed, 23 Jul 2025 12:23:04 GMT, Matthias Baesken wrote: > Older gcc version (8, 9) do not support the -mtune=power10 flag. We should > set this only when recent gcc versions are used. Very good, this helps our CI errors. /Thomas - Marked as reviewed by stuefe (Reviewer). PR Re