Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Correct typos mentioning classDepthAndFlags field #7330

Merged

Conversation

hzongaro
Copy link
Contributor

@hzongaro hzongaro commented May 9, 2024

Code in OMR supports the JIT compiler in generating IL to access the J9Class field classDepthAndFlags in the downstream OpenJ9 project. In many cases, that compiler code incorrectly refers to the field as classAndDepthFlags.

This commit adds new duplicate definitions of the methods and enum values to use the correct name. Once the downstream project removes references to the obsolete methods, the obsolete definitions will be removed.

Code in OMR supports the JIT compiler in generating IL to access the
J9Class field classDepthAndFlags field in the downstream OpenJ9 project.
In many cases, that code incorrectly refers to the field as
classAndDepthFlags.

This commit adds new duplicate definitions of the methods and enum
values that correct the name.  Once the downstream project removes
references to the obsolete methods, the obsolete definitions will be
removed.
@hzongaro
Copy link
Contributor Author

hzongaro commented May 9, 2024

@0xdaryl, may I ask you to review this change?

@0xdaryl
Copy link
Contributor

0xdaryl commented May 9, 2024

Jenkins build all

@0xdaryl 0xdaryl self-assigned this May 9, 2024
@0xdaryl 0xdaryl merged commit b9a6ccc into eclipse-omr:master May 9, 2024
15 of 18 checks passed
@hzongaro hzongaro deleted the correct-mentions-of-classDepthAndFlags branch May 10, 2024 13:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants