[flang-commits] [clang] [clang-tools-extra] [flang] [lldb] [llvm] [clang] Refactor option-related code from clangDriver into new clangOptions library (PR #163659)

[Jan Svoboda via flang-commits]

================
@@ -27,7 +27,6 @@
 #include "clang/Basic/XRayInstr.h"
 #include "clang/Config/config.h"
 #include "clang/Driver/Driver.h"
----------------
jansvoboda11 wrote:

I see, thanks. I'm fine with moving this into `clangDriver` as long as it's not the **only** reasons other targets need to link `clangDriver` instead of just `clangOptions`.

On the other hand, in the future, it _could_ force targets into linking `clangDriver` instead of just `clangOptions`, which might be wasteful. So maybe moving to `clangOptions` is the better solution after all?

I'll leave the decision up to you, though.

https://github.com/llvm/llvm-project/pull/163659