r330210 - Remove unused variable

Peter Collingbourne via cfe-commits cfe-commits at lists.llvm.org
Tue Apr 17 13:27:52 PDT 2018


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 at 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 at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
>



-- 
-- 
Peter
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20180417/45454c9b/attachment.html>


More information about the cfe-commits mailing list