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

Teresa Johnson via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Apr 17 13:05:43 PDT 2018


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);
----------------
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.


Repository:
  rL LLVM

https://reviews.llvm.org/D45217





More information about the cfe-commits mailing list