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

Disable the OMR_THR_MCS_LOCKS flag #4660

Merged
merged 1 commit into from
Dec 10, 2019
Merged

Conversation

babsingh
Copy link
Contributor

@babsingh babsingh commented Dec 9, 2019

#4596 was accidentally merged.

Signed-off-by: Babneet Singh sbabneet@ca.ibm.com

eclipse#4596 was accidentally merged.

Signed-off-by: Babneet Singh <sbabneet@ca.ibm.com>
@babsingh
Copy link
Contributor Author

babsingh commented Dec 9, 2019

fyi @rwy0717 This PR is a revert of 94e76ea.

@rwy7
Copy link
Member

rwy7 commented Dec 9, 2019

@genie-omr build all

@youngar
Copy link
Member

youngar commented Dec 10, 2019

PR #4596 was considered functionally correct, but was awaiting performance testing. The new code can be easily disabled by turning off the build flag.

@youngar youngar self-assigned this Dec 10, 2019
@youngar youngar merged commit 522a98f into eclipse:master Dec 10, 2019
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