[PATCH] D9509: [cuda] Driver changes to build and stitch together host and device-side CUDA code.

[Eric Christopher]

echristo accepted this revision.
echristo added a comment.
This revision is now accepted and ready to land.

This is a good start. We'll probably want to do some more refactoring, but this gets it in so we can look at it and make those decisions.

Make sure everything is formatted with clang-format, then LGTM.

-eric


================
Comment at: lib/Driver/Driver.cpp:822-823
@@ -820,3 +821,4 @@
 
 static unsigned PrintActions1(const Compilation &C, Action *A,
-                              std::map<Action*, unsigned> &Ids) {
+                              std::map<Action *, unsigned> &Ids);
+
----------------
tra wrote:
> echristo wrote:
> > Can you rearrange the code so that it doesn't need the forward declaration?
> Nope. PrintActions1 and PrintActionList call each other recursively. At least one of them must have forward declaration.
Bleh. OK.

================
Comment at: lib/Driver/ToolChains.cpp:3668-3670
@@ +3667,5 @@
+
+llvm::opt::DerivedArgList *
+CudaToolChain::TranslateArgs(const llvm::opt::DerivedArgList &Args,
+                    const char *BoundArch) const {
+  DerivedArgList *DAL = new DerivedArgList(Args.getBaseArgs());
----------------
Formatting?


http://reviews.llvm.org/D9509