[foreign-jextract] RFR: 8277645: TreeMaker should not call type() on pointer types too early

[Jorn Vernee]

On Tue, 23 Nov 2021 14:05:34 GMT, Maurizio Cimadamore <mcimadamore at openjdk.org> wrote:

> This patch fixes a couple of callsites in TreeMaker which were calling `DelegatedType::type`.
> 
> `DelegatedType` is a common supertype for both typedef types and pointer types. The reason why `TreeMaker` calls that method is to recover the *canonical* type of a typedef.
> 
> Unfortunately, as `TreeMaker` doesn't check the delegated type kind, it can sometime call `type()` on a delegated type whose kind is `POINTER`. The way in which the jextract parser is setup is such that pointee types can only be accessed *after* parsing has completed.
> 
> This issue only occurs under very rare circumstances, that is when a type qualifier (other than the usual ones, such as `const`) is used on a pointer typedef - such as `__ptr64`. In such cases, the clang AST sees an `AttributedType` and not a `QualifiedType`, which throws our parsing logic off-guard.
> 
> The solution is to check for the delegated type kind before attempting to call `type()`, and only call it when its kind is `TYPEDEF`. To test this I used the special clang flag `-fms-extensions` which allows to use the MS extensions even on Linux/Mac.

test/jdk/tools/jextract/TestAttributedPointerTypedef.java line 29:

> 27:  * @library /test/lib
> 28:  * @build BadBitfieldTest
> 29:  * @run testng/othervm --enable-native-access=jdk.incubator.jextract BadBitfieldTest

This is not running the right test class it seems.

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

PR: https://git.openjdk.java.net/panama-foreign/pull/616