tra added a subscriber: echristo. tra added a comment. This looks like fix-includes and it may be somewhat shaky if users start messing with include paths. You may want to get @echristo's input on that. I' personally would prefer to force-include these files. I suspect it will not change things much as we already include a lot.
================ Comment at: clang/lib/Driver/ToolChains.cpp:4704 @@ +4703,3 @@ + llvm::sys::path::append(P, "include"); + llvm::sys::path::append(P, "cuda_wrappers"); + addSystemInclude(DriverArgs, CC1Args, P); ---------------- path::append accepts multiple path parts so you can construct path in one call. https://reviews.llvm.org/D24979 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
