Skip to content

LLVM+Clang: only include what we actually need - #597

Closed
agarny wants to merge 11 commits into
opencor:mainfrom
agarny:llvm+clang
Closed

agarny wants to merge 11 commits into
opencor:mainfrom
agarny:llvm+clang

Conversation

@agarny

@agarny agarny commented Sep 24, 2026

Copy link
Copy Markdown
Contributor

No description provided.

Make sure that `PACKAGE_C_FLAGS` is a single string (and not a list) so that it can be passed to `build_package()`, which would otherwise insert semicolons between the flags.
This is particularly useful when loading Web OpenCOR since it dynamically imports libOpenCOR's WASM. So, the smaller it is the better the user experience.
Copilot AI lite review requested due to automatic review settings September 24, 2026 07:14
@agarny agarny closed this Sep 24, 2026

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

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.

2 participants