Hahnfeld added inline comments.
================ Comment at: lib/Frontend/CompilerInvocation.cpp:1044-1045 - Opts.CudaGpuBinaryFileNames = - Args.getAllArgValues(OPT_fcuda_include_gpubinary); + Opts.CudaGpuBinaryFileName = + Args.getLastArgValue(OPT_fcuda_include_gpubinary); ---------------- 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...) Repository: rC Clang https://reviews.llvm.org/D43461 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
