-
Notifications
You must be signed in to change notification settings - Fork 738
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
Convert TR_J9SharedCache APIs in Optimizer to fej9 APIs #4668
Conversation
3e6140e
to
26cc195
Compare
@dsouzai Hi Irwin, I'm wondering if you could review this? |
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 the #include "env/SharedCache.hpp"
in both InlinerTempForJ9.cpp
and J9Inliner.cpp
.
26cc195
to
a47307b
Compare
change updated. |
You'll have to fix the copyrights; I'll hold off kicking off the jenkins tests until then. |
The purpose of this change is to hide away the TR_J9SharedCache APIs that were called in the Optimizer so that the Optimizer doesn't need to know about the TR_J9SharedCache object. The Optimizer can query about the TR_J9SharedCache object through J9 frontend APIs. Signed-off-by: Harry Yu <harryyu1994@gmail.com>
a47307b
to
7881873
Compare
copyright fixed. |
jenkins test sanity all jdk11 |
🤷♂️ |
jenkins test sanity zlinux jdk11 |
The purpose of this change is to hide away the TR_J9SharedCache APIs that are called in the Optimizer so that the Optimizer doesn't need to know about the TR_J9SharedCache object. The Optimizer can query information about the TR_J9SharedCache object through J9 frontend APIs.
Signed-off-by: Harry Yu harryyu1994@gmail.com