FWIW, I don't think you actually need the if. Peter
On Tue, Apr 17, 2018 at 1:21 PM, Teresa Johnson via cfe-commits < cfe-commits@lists.llvm.org> wrote: > Author: tejohnson > Date: Tue Apr 17 13:21:53 2018 > New Revision: 330210 > > URL: http://llvm.org/viewvc/llvm-project?rev=330210&view=rev > Log: > Remove unused variable > > Fixes unused variable error introduced in r330194. > > Modified: > cfe/trunk/lib/Driver/ToolChains/Clang.cpp > > Modified: cfe/trunk/lib/Driver/ToolChains/Clang.cpp > URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Driver/ > ToolChains/Clang.cpp?rev=330210&r1=330209&r2=330210&view=diff > ============================================================ > ================== > --- cfe/trunk/lib/Driver/ToolChains/Clang.cpp (original) > +++ cfe/trunk/lib/Driver/ToolChains/Clang.cpp Tue Apr 17 13:21:53 2018 > @@ -3267,7 +3267,7 @@ void Clang::ConstructJob(Compilation &C, > Args.AddLastArg(CmdArgs, options::OPT_fthinlto_index_EQ); > } > > - if (const Arg *A = Args.getLastArg(options::OPT_save_temps_EQ)) > + if (Args.getLastArg(options::OPT_save_temps_EQ)) > Args.AddLastArg(CmdArgs, options::OPT_save_temps_EQ); > > // Embed-bitcode option. > > > _______________________________________________ > cfe-commits mailing list > cfe-commits@lists.llvm.org > http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits > -- -- Peter
_______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits