================ @@ -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 _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
