[clang] [Driver] Print C++ standard library info (PR #207798)

Yaxun Liu via cfe-commits cfe-commits at lists.llvm.org
Wed Jul 29 08:50:20 PDT 2026


yxsamliu wrote:

@trcrsired Re: PR #96417 interaction

Thanks for pointing this out. I was not aware of PR #96417 when I made this change.

The two PRs use `AddClangCXXStdlibIncludeArgs` differently. PR #96417 uses it during normal `clang++ --sysroot=...` compilation to add the C++ header paths. My change uses it to report the standard MSVC include paths for `--print-cxx-stdlib-include-dirs`. Keeping only either implementation will break the other case.

I think PR #96417 should keep its current logic when `--sysroot` is present and use `AddMSVCStdlibIncludeArgs` for the print option when there is no custom sysroot. The existing MSVC tests in `print-cxx-stdlib.cpp` should also be kept. I will be happy to help review PR #96417 so it can accommodate the merged change and support both layouts.

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


More information about the cfe-commits mailing list