Skip to content

Conversation

@tajila
Copy link
Contributor

@tajila tajila commented Nov 18, 2025

Cache the constantPool object off the j.l.Class after the first invocation of Access::getConstantPool. Given that the ConstantPool instance is associated with a single version of the class, as soon as the class is redefined the cache is invalidated and disabled.

Cache the constantPool object off the j.l.Class after the first
invocation of Access::getConstantPool. Given that the ConstantPool
instance is associated with a single version of the class, as soon as
the class is redefined the cache is invalidated and disabled.

Signed-off-by: tajila <[email protected]>
@tajila tajila requested a review from gacholio November 18, 2025 21:43
@tajila
Copy link
Contributor Author

tajila commented Nov 18, 2025

Yet to reproduce the issue on xlinux, so maybe there is a non obvious platform dependency here.

@tajila
Copy link
Contributor Author

tajila commented Nov 18, 2025

jenkins test sanity.openjdk amac jdk21

* constantpool associated with clazz
*/
public static ConstantPool getConstantPoolFromAnnotationBytes(Class<?> clazz, byte[] array) {
/* Check the cp cache on the Class object first */
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please finish each sentence with a period (throughout).

@tajila
Copy link
Contributor Author

tajila commented Nov 19, 2025

@gacholio
Copy link
Contributor

What's different in this one?

@tajila
Copy link
Contributor Author

tajila commented Nov 19, 2025

What's different in this one?

Nothing, Im just trying to reproduce the failure with more debug info.

@tajila
Copy link
Contributor Author

tajila commented Nov 20, 2025

jenkins test sanity.openjdk amac jdk21

@tajila
Copy link
Contributor Author

tajila commented Nov 20, 2025

jenkins test sanity.openjdk amac jdk21

@tajila
Copy link
Contributor Author

tajila commented Nov 21, 2025

jenkins test sanity.openjdk amac jdk21

@tajila
Copy link
Contributor Author

tajila commented Nov 21, 2025

jenkins test sanity.openjdk amac jdk21

@tajila
Copy link
Contributor Author

tajila commented Nov 21, 2025

jenkins test sanity.openjdk amac jdk21

@tajila
Copy link
Contributor Author

tajila commented Nov 21, 2025

jenkins test sanity plinux jdk17

@tajila tajila force-pushed the opt branch 3 times, most recently from eaa45af to 4facbef Compare November 24, 2025 15:24
@tajila
Copy link
Contributor Author

tajila commented Nov 24, 2025

jenkins test sanity.openjdk amac jdk21

@tajila tajila force-pushed the opt branch 2 times, most recently from 6cbee62 to 7736698 Compare November 24, 2025 22:57
@tajila
Copy link
Contributor Author

tajila commented Nov 24, 2025

jenkins test sanity.openjdk amac jdk21

@tajila
Copy link
Contributor Author

tajila commented Nov 25, 2025

jenkins test sanity.openjdk amac jdk21

@tajila
Copy link
Contributor Author

tajila commented Nov 26, 2025

jenkins test sanity.openjdk amac jdk21

@tajila tajila force-pushed the opt branch 4 times, most recently from 2e7a389 to 2db6b17 Compare November 27, 2025 15:12
@tajila
Copy link
Contributor Author

tajila commented Nov 27, 2025

jenkins test sanity.openjdk amac jdk21 depends eclipse-omr/omr#8060

Signed-off-by: Tobi Ajila <[email protected]>
@tajila
Copy link
Contributor Author

tajila commented Nov 28, 2025

jenkins test sanity.openjdk amac jdk21 depends eclipse-omr/omr#8060

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants