Bug#1057838: clang-17: c++ include dir from sysroot unexpectedly in include paths when in wasm c mode

[Faidon Liambotis]

Thanks, Sylvestre for the Cc and Israel for the bug report!

On Fri, Dec 15, 2023 at 02:00:55PM +0100, Sylvestre Ledru wrote:
> > The above lines are adding c++ include paths in the
> > method (WebAssembly::AddClangSystemIncludeArgs)
> > for adding only system/c include paths.
> > I think the c++ include paths should only be added in the methods for
> > adding
> > libc++ (WebAssembly::addLibCxxIncludePaths) or libstdc++
> > (WebAssembly::addLibStdCXXIncludePaths) include
> > paths.

Israel, I can confirm that your interpetation is correct :) Can you
confirm whether you're experiencing the same issue with clang-16?

Sylvestre, from a cursory look I think the patch that I recently
provided, the one that you applied and was included in the
src:llvm-toolchain-16 1:16.0.6-19 upload, has not been carried over to
at least the 17 branch. I think this will probably fix this bug as well.

Thanks,
Faidon