RFR: 8352075: Perf regression accessing fields [v4]

Radim Vansa rvansa at openjdk.org
Mon May 12 06:26:53 UTC 2025


On Fri, 9 May 2025 21:22:15 GMT, Coleen Phillimore <coleenp at openjdk.org> wrote:

> ```
> 84	      assert(compare_symbols(fields->adr_at(i - 1)->name(constants), fields->adr_at(i)->name(constants)) < 0, "Fields should be sorted");
> ```
> 
> Happens with names matching (not signatures). How does the same name for two fields work if it spans a JUMP_TABLE_STRIDE?
> 
> This should add some tests.

The stream contains 'declared fields', not including superclass... How could two names match?

https://docs.oracle.com/javase/specs/jls/se17/html/jls-8.html#jls-8.3

> It is a compile-time error for the body of a class declaration to declare two fields with the same name. 

Is this some generated code that breaks JLS?

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

PR Comment: https://git.openjdk.org/jdk/pull/24847#issuecomment-2871010896


More information about the hotspot-dev mailing list