[PATCH] D43461: [CUDA] Include single GPU binary, NFCI.

[Artem Belevich via Phabricator via cfe-commits]

tra accepted this revision.
tra added inline comments.
This revision is now accepted and ready to land.


================
Comment at: lib/Frontend/CompilerInvocation.cpp:1044-1045
 
-  Opts.CudaGpuBinaryFileNames =
-      Args.getAllArgValues(OPT_fcuda_include_gpubinary);
+  Opts.CudaGpuBinaryFileName =
+      Args.getLastArgValue(OPT_fcuda_include_gpubinary);
 
----------------
Hahnfeld wrote:
> tra wrote:
> > If more than one gpu binary is passed, all but last will be ignored.
> > IMO in this case we would want to either warn that some inputs were ignored or report an error that there is more than one GPU binary.
> Well, `-fcuda-include-gpubinary` is only recognized on `cc1`. I think we can assume that we are correctly assembling our command line, can't we? (Nobody else checks the options here...)
Fair enough. Assert, then?


Repository:
  rC Clang

https://reviews.llvm.org/D43461