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

Add options to disable inlining Thread.currentThread #6950

Merged

Conversation

a7ehuo
Copy link
Contributor

@a7ehuo a7ehuo commented Apr 13, 2023

Add option disableRecognizeCurrentThread to disable inlining Thread.currentThread()

@a7ehuo a7ehuo requested review from 0xdaryl and mstoodle as code owners April 13, 2023 15:49
Add option `disableRecognizeCurrentThread` to disable
inlining `Thread.currentThread()`.

Signed-off-by: Annabelle Huo <Annabelle.Huo@ibm.com>
@a7ehuo a7ehuo force-pushed the add-option-disableRecognizeCurrentThread branch from 99fdd69 to ab77a7a Compare April 13, 2023 15:52
@a7ehuo a7ehuo removed the request for review from mstoodle April 13, 2023 15:52
@a7ehuo a7ehuo added comp:compiler new platform Cross component work needed to support a new platform and removed new platform Cross component work needed to support a new platform labels Apr 13, 2023
@a7ehuo
Copy link
Contributor Author

a7ehuo commented Apr 13, 2023

@0xdaryl May I ask you to review this change that adds an option disableRecognizeCurrentThread to disable inlining Thread.currentThread()? Thanks!

The OpenJ9 change that uses this option is in eclipse-openj9/openj9#17182

@0xdaryl
Copy link
Contributor

0xdaryl commented Apr 14, 2023

Jenkins build all

@0xdaryl 0xdaryl self-assigned this Apr 14, 2023
@0xdaryl 0xdaryl merged commit 5ba6dd5 into eclipse-omr:master Apr 14, 2023
@a7ehuo a7ehuo deleted the add-option-disableRecognizeCurrentThread branch March 6, 2024 16:32
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