[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