Skip to content

[build-script][embedded] Conditionally force compiler-rt baremetal builtins #77395

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Conversation

kastiglione
Copy link
Contributor

@kastiglione kastiglione commented Nov 5, 2024

This change allows build-script to be invoked with the following combination:

--llvm-targets-to-build=host --build-embedded-stdlib=false

When --build-embedded-stdlib=false is used, it's unnecessary to force-configure
additional compiler-rt targets. Further, when --llvm-targets-to-build is used,
the resulting llvm usually won't support the force targets of armv6/armv7.

…iltins

This change allows build-script to be invoked with the following combination:

```
--llvm-targets-to-build=host --build_embedded_stdlib=false
```

When `--build-embedded-stdlib=false` is used, it's unnecessary to force-configure
additional compiler-rt targets. Further, when `--llvm-targets-to-build` is used,
the resulting llvm usually won't support the force targets of armv6/armv7.
@kastiglione
Copy link
Contributor Author

@swift-ci smoke test

@kastiglione kastiglione enabled auto-merge (squash) November 5, 2024 18:07
@kastiglione
Copy link
Contributor Author

@swift-ci smoke test windows

@kastiglione kastiglione merged commit b8db9d4 into main Nov 5, 2024
3 checks passed
@kastiglione kastiglione deleted the dl/build-script-embedded-Conditionally-force-compiler-rt-baremetal-builtins branch November 5, 2024 23:30
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