[PATCH] D45217: [ThinLTO] Pass -save-temps to LTO backend for distributed ThinLTO builds

[Teresa Johnson via Phabricator via llvm-commits]

tejohnson added inline comments.


================
Comment at: cfe/trunk/lib/Driver/ToolChains/Clang.cpp:3270
 
+  if (const Arg *A = Args.getLastArg(options::OPT_save_temps_EQ))
+    Args.AddLastArg(CmdArgs, options::OPT_save_temps_EQ);
----------------
tejohnson wrote:
> bjope wrote:
> > A is not used, so this does not compile when using -Werror:
> > 
> > ```
> > ../tools/clang/lib/Driver/ToolChains/Clang.cpp:3270:18: error: unused variable 'A' [-Werror,-Wunused-variable]
> >   if (const Arg *A = Args.getLastArg(options::OPT_save_temps_EQ))
> > 
> > ```
> Sorry, will fix shortly.
Fixed in r330210


Repository:
  rL LLVM

https://reviews.llvm.org/D45217