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
An improved check for ignoring the c2-crash test if running on a client compiler. #12953
Conversation
FYI, there's also |
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.
Remove this method, too (it is dead code):
This is why it works and you can still run the tests with
This PR just detects directly if Client VM is used. So it doesn't matter if it is caused by CI enabled or if you run it in your IDE with some custom config. |
I've removed the unused method and added hotspot check, thanks Uwe. |
I've applied this to main and branch_9x. |
I'm way too slow here, sorry. Belated LGTM. And thanks for following up on this. |
Follow-up to #12905