Update border frequencies under server inlining - #22561
Conversation
dsouzai
added
comp:jit
depends:omr
|
Looks okay to me. I will await Marius' review as well |
|
Please correct the two typos in "frquences" in the commit message (and PR title) and I can start testing. |
dsouzai
changed the title
|
That commit only fixed one of the typos :) "frequences" -> "frequencies" is still needed |
dsouzai
changed the title
This change has been shown to help improve throughput without impacting rampup. Signed-off-by: Irwin D'Souza <dsouzai.gh@gmail.com>
|
jenkins test sanity all jdk21 |
|
jenkins test sanity all jdk21 depends eclipse-omr/omr#7924 |
|
jenkins test sanity amac jdk21 |
|
jenkins test sanity xmac jdk21 |
|
amac nodes are up, but the build node is still offline; seems like xmac nodes are all up so I kicked off testing for that. |
|
jenkins test sanity xmac jdk21 depends eclipse-omr/omr#7924 |
1 similar comment
|
jenkins test sanity xmac jdk21 depends eclipse-omr/omr#7924 |
|
nvm, looks like xmac build machine are also offline. |
|
@vijaysun-omr The dependent OMR changes have propagated. However, I think the mac nodes are still offline. Because this is common code and all platforms have succeeded including aarch linux and x linux, this should be safe to merge. |
|
I am going to merge given the last comment and the fact that it is a common code, heuristic change at it's core, rather than adding new function as such, and it passed on other platforms. |
3 participants
Update the border frequencies under isServerInlining to 500. This change has been shown to help improve throughput without impacting rampup.
Depends on eclipse-omr/omr#7924