Skip to content

We do not need disableOSRGuardsMerging to enforceVectorAPIExpansion - #6828

Merged
vijaysun-omr merged 1 commit into
eclipse-omr:masterfrom
r30shah:removeOption
Nov 29, 2022
Merged

We do not need disableOSRGuardsMerging to enforceVectorAPIExpansion#6828
vijaysun-omr merged 1 commit into
eclipse-omr:masterfrom
r30shah:removeOption

Conversation

@r30shah

@r30shah r30shah commented Nov 28, 2022

Copy link
Copy Markdown
Contributor

Since we have made improvements to allow marking guards for the Vector JEP method does not kill fear, we no longer need to disable merging of OSR guards to enforce Vector API Expansion. Remove setting that option.

Signed-off-by: Rahil Shah rahil@ca.ibm.com

Since we have made improvements to allow marking guards for the Vector
JEP method does not kill fear, we no longer need to disable merging of
OSR guards to enforce Vector API Expansion. Remove setting that option.

Signed-off-by: Rahil Shah <rahil@ca.ibm.com>
@r30shah

r30shah commented Nov 28, 2022

Copy link
Copy Markdown
Contributor Author

With the changes in eclipse-openj9/openj9#16354 getting merged soon, we no longer need to set disableOSRGuardsMerging to enforce expansion.
@gita-omr / @vijaysun-omr Can I please get a review please?

@vijaysun-omr

Copy link
Copy Markdown
Contributor

Jenkins build all

@gita-omr

Copy link
Copy Markdown
Contributor

I think this PR (merged) eclipse-openj9/openj9#16206 already eliminates the need for disableOSRGuardsMerging.

@vijaysun-omr

Copy link
Copy Markdown
Contributor

Only failing test is the known/unrelated Power Linux LE failure. Merging.

@vijaysun-omr
vijaysun-omr merged commit 145cf4b into eclipse-omr:master Nov 29, 2022
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.

3 participants