Skip to content

[clang] Move CCC_OVERRIDE_OPTIONS implementation to Driver #8522

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

Move CCC_OVERRIDE_OPTIONS support to clangDriver so that it may be used outside of the
clang driver binary.

The override functionality will be used in LLDB, to apply adjustments to ClangImporter
flags. This will be useful as an escape hatch when there are issues that can be fixed
by adding or removing clang flags.

The only thing changed is the name, from ApplyQAOverride to applyOverrideOptions.

(cherry picked from commit 8d7ee46)

(cherry-picked from commit f8ce350)

Move CCC_OVERRIDE_OPTIONS support to clangDriver so that it may be used outside of the
clang driver binary.

The override functionality will be used in LLDB, to apply adjustments to ClangImporter
flags. This will be useful as an escape hatch when there are issues that can be fixed
by adding or removing clang flags.

The only thing changed is the name, from `ApplyQAOverride` to `applyOverrideOptions`.

(cherry picked from commit 8d7ee46)

(cherry-picked from commit f8ce350)
@kastiglione kastiglione changed the title [clang] Move CCC_OVERRIDE_OPTIONS implementation to Driver (#85425) [clang] Move CCC_OVERRIDE_OPTIONS implementation to Driver Apr 2, 2024
@kastiglione
Copy link
Author

@swift-ci test

@adrian-prantl adrian-prantl merged commit 06b2847 into swift/release/6.0 Apr 4, 2024
@kastiglione kastiglione deleted the dl/swift/release/6.0/clang-Move-CCC_OVERRIDE_OPTIONS-implementation-to-Driver-85425 branch April 4, 2024 21:02
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