Skip to content

Conversation

@graalvmbot
Copy link
Collaborator

This contains bugfixes for issues discovered by the tests in the attached enterprise PR, namely:

  • Added missing agent breakpoint for Class.forName(Module, String);
  • Added missing registration error (behind throwMissingRegistrationErrors()) and missing agent breakpoint for Class.arrayType();
  • Registering fields for reflection was done at query time, so this PR changes it to be detected at access time (in Field.get/set) and adds the necessary agent breakpoints. This also enables a simplification of the ReflectionProcessor;
  • Invoking a field access method through Method.invoke now correctly registers the accessed field;
  • Added missing AccessAdvisor entry for LoggingMXBeanAccess;
  • Use unchecked array instantiation for metadata decoding at run-time, avoiding missing registration checks in Array.newInstance;
  • Use unchecked getArrayHub() for metadata tracing in Array.newArray, avoiding an infinite loop when using the native tracing mode;
  • Fixed the DynamicAccessDetectionPhase to handle inner classes correctly (the $ in the class name was being interpreted as a variable in the generated shell script)

@oracle-contributor-agreement oracle-contributor-agreement bot added the OCA Verified All contributors have signed the Oracle Contributor Agreement. label Jul 31, 2025
@graalvmbot graalvmbot merged commit 01f68fc into master Jul 31, 2025
13 checks passed
@graalvmbot graalvmbot deleted the lottet/GR-67746-tck-reflection-base branch July 31, 2025 09:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

OCA Verified All contributors have signed the Oracle Contributor Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants