================
@@ -33,32 +33,22 @@ using namespace llvm::opt;
#define NULL_FILE "/dev/null"
#endif
-void AMDGCN::Linker::constructLlvmLinkCommand(Compilation &C,
- const JobAction &JA,
- const InputInfoList &Inputs,
- const InputInfo &Output,
- const llvm::opt::ArgList &Args) const
{
- // Construct llvm-link command.
- // The output from llvm-link is a bitcode file.
- ArgStringList LlvmLinkArgs;
+void AMDGCN::Linker::constructLlvmLinkCommand(
+ Compilation &C, const JobAction &JA, const InputInfoList &Inputs,
+ const InputInfo &Output, const llvm::opt::ArgList &Args) const {
- assert(!Inputs.empty() && "Must have at least one input.");
----------------
mgcarrasco wrote:
Wouldn't it better to keep the assertion? The function should not change its
behaviour with these new changes.
https://github.com/llvm/llvm-project/pull/169572
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
