RFR: 7904115: Fix for AIX test case failures due to incorrect alignment for double and pointer [v4]
Varada M
varadam at openjdk.org
Fri Dec 12 15:41:16 UTC 2025
On Fri, 12 Dec 2025 15:27:56 GMT, Varada M <varadam at openjdk.org> wrote:
>> Total of 10 test failures observed on AIX:
>> jtreg/generator/nestedTypes/TestNestedTypesUnsupported.java
>> jtreg/generator/test8246400/LibTest8246400Test.java
>> jtreg/generator/test8258605/LibTest8258605Test.java
>> jtreg/generator/test8261511/Test8261511.java
>> jtreg/generator/testStruct/LibStructTest.java
>> testng/org/openjdk/jextract/test/toolprovider/ConstantsTest.java
>> testng/org/openjdk/jextract/test/toolprovider/IncompleteArrayTest.java
>> testng/org/openjdk/jextract/test/toolprovider/Test8240811.java
>> testng/org/openjdk/jextract/test/toolprovider/TestClassGeneration.java
>> testng/org/openjdk/jextract/test/toolprovider/nestedAnonOffset/TestNestedAnonOffset.java
>>
>> This PR fixes AIX specific layout generation issues related to incorrect alignment double and pointer types.
>> 1. Structs containing double fields fail with:
>> i. Unsupported layout: 4%D8
>> ii. Invalid alignment constraint for member layout
>> double in AIX structs has size 8 but alignment 4 (except as first field). AIX specific handling for C_DOUBLE computes the correct alignment.
>>
>> 2. Clang was detected as 32-bit due to missing -m64 during macro extraction, causing inconsistent macros. This caused jextract to interpret pointer constants incorrectly, leading to failures like:
>> expected [-1] but found [4294967295]
>>
>> 3. TestNestedAnonOffset.java test failed on AIX because it also expects more padding similar to platforms like windows and linux
>>
>>
>> After the patch jtreg tests passes successfully.
>>
>> JBS: [CODETOOLS-7904115](https://bugs.openjdk.org/browse/CODETOOLS-7904115)
>
> Varada M has updated the pull request incrementally with one additional commit since the last revision:
>
> added field alignment and natural alignment argument
The test failure "Invalid alignment constraint for member layout: D8(d)" on AIX occurs because the default alignment mode is power where the subsequent double fields is 4 byte aligned, but the generated double is D8, 8 byte aligned.
Using the natural alignment mode resolves the test failures
-------------
PR Comment: https://git.openjdk.org/jextract/pull/296#issuecomment-3647059054
More information about the jextract-dev
mailing list