gtbercea marked an inline comment as done. gtbercea added inline comments.
================ Comment at: lib/Driver/ToolChains/Cuda.cpp:707 CmdArgs.push_back(StaticLibName); + } else if (OrigInputFileName.endswith(".so")) { + // Only static libraries are supported. ---------------- ABataev wrote: > gtbercea wrote: > > ABataev wrote: > > > I would suggest to use `types::lookupTypeForExtension` function and > > > compare the result with the `types::TY_Object`. > > Isn't that too broad? I only want .so files. > I don't think nvlink supports anything else except for objects. Also, in > future, we could extend it if it is required. But won't this exclude .o files too? I don't want to ignore those. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D59028/new/ https://reviews.llvm.org/D59028 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits