[lworld] Integrated: 8353639: [lworld] IGV: Fix broken type info filter with TypeAryPtr::_field_offset
Christian Hagedorn
chagedorn at openjdk.org
Thu Apr 3 12:29:37 UTC 2025
On Thu, 3 Apr 2025 12:19:17 GMT, Christian Hagedorn <chagedorn at openjdk.org> wrote:
> The `TypeAryPtr::_field_offset` information is dumped inside a pair of braces:
>
> --> HERE <--
> flat:narrowoop: V1:NotNull:exact *[int:≥0]:NotNull:exact:flat(+4):null_free[2] * [narrow]
>
> The problem is that the type info filter from mainline only expects that interface types are found inside braces and then tries to remove them. This now messes with two pair of braces.
>
> The fix I propose is to match anything but the first closing `)` when finding the first `(` with `([^)]*)` for the interface types instead of matching it with `(.*)` which matches up to the field offset.
>
> Without fix:
> 
>
> With fix:
> 
>
> Thanks,
> Christian
This pull request has now been integrated.
Changeset: f662357d
Author: Christian Hagedorn <chagedorn at openjdk.org>
URL: https://git.openjdk.org/valhalla/commit/f662357d3a5b78aa7f89ce81a04f39185a3d233a
Stats: 1 line in 1 file changed: 0 ins; 0 del; 1 mod
8353639: [lworld] IGV: Fix broken type info filter with TypeAryPtr::_field_offset
Reviewed-by: thartmann
-------------
PR: https://git.openjdk.org/valhalla/pull/1414
More information about the valhalla-dev
mailing list