llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT--> @llvm/pr-subscribers-clang @llvm/pr-subscribers-clang-driver Author: Joseph Huber (jhuber6) <details> <summary>Changes</summary> Summary: These command line invocations can become so large that they no longer fit, we should support response files in this case so the build on Windows can be unblocked with the new driver. --- Full diff: https://github.com/llvm/llvm-project/pull/183548.diff 1 Files Affected: - (modified) clang/lib/Driver/ToolChains/Clang.cpp (+1-1) ``````````diff diff --git a/clang/lib/Driver/ToolChains/Clang.cpp b/clang/lib/Driver/ToolChains/Clang.cpp index 79005d97b41ee..1b73fde5f5c25 100644 --- a/clang/lib/Driver/ToolChains/Clang.cpp +++ b/clang/lib/Driver/ToolChains/Clang.cpp @@ -9240,7 +9240,7 @@ void OffloadPackager::ConstructJob(Compilation &C, const JobAction &JA, } C.addCommand(std::make_unique<Command>( - JA, *this, ResponseFileSupport::None(), + JA, *this, ResponseFileSupport::AtFileUTF8(), Args.MakeArgString(getToolChain().GetProgramPath(getShortName())), CmdArgs, Inputs, Output)); } `````````` </details> https://github.com/llvm/llvm-project/pull/183548 _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
