[clang] [clang-tools-extra] [flang] [lldb] [llvm] [clang] Move options from clangDriver into new clangOptions library (NFC) (PR #163659)
Vlad Serebrennikov via cfe-commits
cfe-commits at lists.llvm.org
Fri Oct 31 02:33:27 PDT 2025
https://github.com/Endilll commented:
> The goal of this change is to remove dependencies on the Driver.
This part of PR description led me to believe that you're going to replace some `clangDriver` dependencies with `clangOptions`, but I only saw you adding `clangOptions` to various CMakeLists. I guess description can be improved here (or my understanding, if I missed something).
It seems that this PR is requiring downstream users who link with `clangDriver` to also link with `clangOptions`. Because of this, I don't consider this to be an NFC change, and would like to see a release note in potentially breaking changes.
I'd like @AaronBallman to sign this off before it's merged. He should be back next week.
https://github.com/llvm/llvm-project/pull/163659
More information about the cfe-commits
mailing list