Skip to content

Add -threads to web_ and gui_ targets#4313

Merged
maliberty merged 1 commit into
The-OpenROAD-Project:masterfrom
The-OpenROAD-Project-staging:web-gui-threads
Jun 25, 2026
Merged

Add -threads to web_ and gui_ targets#4313
maliberty merged 1 commit into
The-OpenROAD-Project:masterfrom
The-OpenROAD-Project-staging:web-gui-threads

Conversation

@maliberty

Copy link
Copy Markdown
Member

Web takes advantage of threading.

Web takes advantage of threading.

Signed-off-by: Matt Liberty <mliberty@precisioninno.com>
@maliberty maliberty self-assigned this Jun 25, 2026

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request updates flow/scripts/variables.mk to append the -threads $(NUM_CORES) argument to both OPENROAD_GUI_CMD and OPENROAD_WEB_CMD. This ensures that multi-threading is enabled consistently for GUI and web modes. There are no review comments, and I have no further feedback to provide.

@maliberty maliberty merged commit 0830009 into The-OpenROAD-Project:master Jun 25, 2026
5 of 8 checks passed
@maliberty maliberty deleted the web-gui-threads branch June 25, 2026 16:55
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.

1 participant