[PATCH] D22518: Refactor how include paths are appended to the command arguments.
Samuel Antao via cfe-commits
cfe-commits at lists.llvm.org
Thu Jul 28 15:52:51 PDT 2016
sfantao added a comment.
In https://reviews.llvm.org/D22518#500066, @tra wrote:
> Samuel, the patch breaks CUDA. With the patch clang no longer adds include paths to standard c++ library during device-side compilation.
> if you run "clang++ -### -c -x cuda /dev/null" you will see that host side gets `"-internal-isystem" "/usr/lib/gcc/x86_64-linux-gnu/4.8/../../../../include/x86_64-linux-gnu/c++/4.8"` but device-side compilation does not.
Hi Art,
The enumerators were switched... It should be fixed in r277064. I'll post a patch with regression test to catch the issue.
Sorry for the trouble,
Samuel
https://reviews.llvm.org/D22518
More information about the cfe-commits
mailing list