Skip to content

Remove some uses of TR::comp() - #16201

Merged
dsouzai merged 11 commits into
eclipse-openj9:masterfrom
0xdaryl:removecomp_20221022
Oct 26, 2022
Merged

Remove some uses of TR::comp()#16201
dsouzai merged 11 commits into
eclipse-openj9:masterfrom
0xdaryl:removecomp_20221022

Conversation

@0xdaryl

@0xdaryl 0xdaryl commented Oct 26, 2022

Copy link
Copy Markdown
Contributor

Remove uses of expensive TR::comp() queries. Refactor APIs to pass available Compilation objects where appropriate.

Signed-off-by: Daryl Maier <maier@ca.ibm.com>
Refactor API to accept a TR::Compilation object directly.

Signed-off-by: Daryl Maier <maier@ca.ibm.com>
Refactor API to accept a TR::Compilation object directly.

Signed-off-by: Daryl Maier <maier@ca.ibm.com>
Refactor API to accept a TR::Compilation object directly.

Signed-off-by: Daryl Maier <maier@ca.ibm.com>
Refactor API to accept a TR::Compilation object directly.

Signed-off-by: Daryl Maier <maier@ca.ibm.com>
Signed-off-by: Daryl Maier <maier@ca.ibm.com>
Refactor API to accept a TR::Compilation object directly.

Signed-off-by: Daryl Maier <maier@ca.ibm.com>
Refactor API to accept a TR::Compilation object directly.

Signed-off-by: Daryl Maier <maier@ca.ibm.com>
Refactor API to accept a TR::Compilation object directly.

Signed-off-by: Daryl Maier <maier@ca.ibm.com>
Refactor API to accept a TR::Compilation object directly.

Signed-off-by: Daryl Maier <maier@ca.ibm.com>
Refactor API to accept a TR::Compilation object directly.

Signed-off-by: Daryl Maier <maier@ca.ibm.com>
@0xdaryl 0xdaryl changed the title Removecomp 20221022 Remove some uses of TR::comp() Oct 26, 2022
@0xdaryl

0xdaryl commented Oct 26, 2022

Copy link
Copy Markdown
Contributor Author

Jenkins test sanity all jdk17

@0xdaryl

0xdaryl commented Oct 26, 2022

Copy link
Copy Markdown
Contributor Author

@dsouzai : would you mind reviewing and merging this cleanup task please?

@dsouzai dsouzai self-assigned this Oct 26, 2022
@dsouzai
dsouzai merged commit ebd4ce3 into eclipse-openj9:master Oct 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants