[PATCH] D24979: [CUDA] Support <complex> and std::min/max on the device.

Artem Belevich via cfe-commits cfe-commits at lists.llvm.org
Tue Sep 27 13:22:40 PDT 2016


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





More information about the cfe-commits mailing list