Skip to content

(0.56) Set node count thresh to 8k for Xtune:throughput - #262

Merged
pshipton merged 1 commit into
eclipse-openj9:v0.56.0-releasefrom
ymanton:nodecount8k-0.56
Oct 7, 2025
Merged

(0.56) Set node count thresh to 8k for Xtune:throughput#262
pshipton merged 1 commit into
eclipse-openj9:v0.56.0-releasefrom
ymanton:nodecount8k-0.56

Conversation

@ymanton

@ymanton ymanton commented Oct 6, 2025

Copy link
Copy Markdown
Member

This change reduces compilation overhead and allows methods to reach higher opt levels sooner.

Port of eclipse-omr/omr#7970.

This change reduces compilation overhead and allows methods to reach
higher opt levels sooner.

Signed-off-by: Younes Manton <ymanton@ca.ibm.com>
@ymanton
ymanton requested a review from vijaysun-omr as a code owner October 6, 2025 21:51
@ymanton

ymanton commented Oct 6, 2025

Copy link
Copy Markdown
Member Author

FYI @vijaysun-omr, @0xdaryl, @pshipton.

@0xdaryl

0xdaryl commented Oct 6, 2025

Copy link
Copy Markdown
Contributor

Normally, we'd let an acceptance build run before merging into a release branch. @pshipton : should we kick one off now rather than wait until 5 am tomorrow?

@0xdaryl

0xdaryl commented Oct 6, 2025

Copy link
Copy Markdown
Contributor

I launched a "lite" OMR Acceptance on all platforms but only for JDK 8 and 17.

@0xdaryl

0xdaryl commented Oct 7, 2025

Copy link
Copy Markdown
Contributor

The AIX acceptance failure appears to be another instance of the long-standing eclipse-openj9/openj9#20156.

Acceptance is clean otherwise.

@pshipton

pshipton commented Oct 7, 2025

Copy link
Copy Markdown
Member

The OMR acceptance didn't test anything new, it ran on the already promoted level.

@0xdaryl

0xdaryl commented Oct 7, 2025

Copy link
Copy Markdown
Contributor

Bah. The 5 am Acceptance Build is looking pretty good so far.

@pshipton
pshipton merged commit a68be5f into eclipse-openj9:v0.56.0-release Oct 7, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants