RFR: 7904079: Fix setting up length parameter for CXUnsavedFiles Struct [v5]
Jorn Vernee
jvernee at openjdk.org
Fri Sep 19 15:09:52 UTC 2025
On Fri, 19 Sep 2025 14:06:16 GMT, Varada M <varadam at openjdk.org> wrote:
>> CXUnsavedFile struct contains three members, first two are pointers and last one is unsigned long. When jextract makes downcall to clang_reparseTranslationUnit_Impl it sets up the third parameter in the struct incorrectly. It sets INT instead of LONG so on big endian platform the parameter will be set differently.
>>
>> JBS Issue : [CODETOOLS-7904079](https://bugs.openjdk.org/browse/CODETOOLS-7904079)
>
> Varada M has updated the pull request incrementally with one additional commit since the last revision:
>
> 7904079: Fix setting up length parameter for CXUnsavedFiles Struct
The latest version passes all tests on my Windows machine.
Could you please update the README at https://github.com/openjdk/jextract/tree/master/updateclang to reflect the changes that were needed to `CXUnsavedFile`? You can include the patch that I sent in that file.
Other than that, this looks ready to me.
-------------
PR Comment: https://git.openjdk.org/jextract/pull/289#issuecomment-3312571016
More information about the jextract-dev
mailing list