-
Notifications
You must be signed in to change notification settings - Fork 392
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
Add OMR Option to disable merging of OSR Guards #6563
Conversation
@gita-omr / @jdmpapin Can I please get your review on this change? I removed the use of environment option to disable merging of OSR guards and added OMR Option so that we can set it through JIT command line option so that it would be easier to test Vector API expansion changes through automated testing. |
LGTM |
Jenkins build all |
compiler/control/OMROptions.cpp
Outdated
@@ -475,6 +475,7 @@ TR::OptionTable OMR::Options::_jitOptions[] = { | |||
#ifdef J9_PROJECT_SPECIFIC | |||
{"disableOSRGuardRemoval", "O\tdisable OSR guard removal", TR::Options::disableOptimization, osrGuardRemoval, 0, "P"}, | |||
#endif | |||
{"disableOSRGuardsMerging", "O\tdisable Merging of OSR guards", SET_OPTION_BIT(TR_DisableOSRGuardsMerging), "F"}, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
To keep the naming consistent with the other OSR guard option on the line above, can you name this "disableOSRGuardMerging" instead?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Previously we had an environment option to disable merging of OSR guards which was needed to allow VectorAPIExpansion to scalarize or vectorize Vector API intrinsics on X and Z. Replacing an enviromentment option to OMR Option to make it easier to set it for automated testing. Signed-off-by: Rahil Shah <rahil@ca.ibm.com>
Jenkins build all |
Previously we had an environment option to disable merging of OSR guards
which was needed to allow VectorAPIExpansion to scalarize or vectorize
Vector API intrinsics on X and Z. Replacing an enviromentment option to
OMR Option to make it easier to set it for automated testing.
Signed-off-by: Rahil Shah rahil@ca.ibm.com