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
