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

Bjorn Pettersson via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Apr 17 12:59:14 PDT 2018


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

```


Repository:
  rL LLVM

https://reviews.llvm.org/D45217





More information about the cfe-commits mailing list