r329445 - Revert "[analyzer] Remove an unused variable"

George Karpenkov via cfe-commits cfe-commits at lists.llvm.org
Mon Apr 9 10:46:46 PDT 2018


Thanks, didn’t know!

> On Apr 9, 2018, at 9:37 AM, David Blaikie <dblaikie at gmail.com> wrote:
> 
> Best if you can use the SubVersion revision number rather than a git hash when reverting. There's a utility in the LLVM project to help with this (llvm/utils/git-svn/git-svnrevert)
> 
> On Fri, Apr 6, 2018 at 12:16 PM George Karpenkov via cfe-commits <cfe-commits at lists.llvm.org <mailto:cfe-commits at lists.llvm.org>> wrote:
> Author: george.karpenkov
> Date: Fri Apr  6 12:14:05 2018
> New Revision: 329445
> 
> URL: http://llvm.org/viewvc/llvm-project?rev=329445&view=rev <http://llvm.org/viewvc/llvm-project?rev=329445&view=rev>
> Log:
> Revert "[analyzer] Remove an unused variable"
> 
> This reverts commit 2fa3e3edc4ed6547cc4ce46a8c79d1891a5b3b36.
> 
> Removed the wrong variable.
> 
> Modified:
>     cfe/trunk/lib/Analysis/LiveVariables.cpp
> 
> Modified: cfe/trunk/lib/Analysis/LiveVariables.cpp
> URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Analysis/LiveVariables.cpp?rev=329445&r1=329444&r2=329445&view=diff <http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Analysis/LiveVariables.cpp?rev=329445&r1=329444&r2=329445&view=diff>
> ==============================================================================
> --- cfe/trunk/lib/Analysis/LiveVariables.cpp (original)
> +++ cfe/trunk/lib/Analysis/LiveVariables.cpp Fri Apr  6 12:14:05 2018
> @@ -381,7 +381,7 @@ void TransferFunctions::VisitBlockExpr(B
>  void TransferFunctions::VisitDeclRefExpr(DeclRefExpr *DR) {
>    const Decl* D = DR->getDecl();
>    bool InAssignment = LV.inAssignment[DR];
> -  if (isa<BindingDecl>(D)) {
> +  if (const auto *BD = dyn_cast<BindingDecl>(D)) {
>      if (!InAssignment)
>        val.liveBindings = LV.BSetFact.add(val.liveBindings, BD);
>    } else if (const auto *VD = dyn_cast<VarDecl>(D)) {
> 
> 
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at lists.llvm.org <mailto:cfe-commits at lists.llvm.org>
> http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits <http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20180409/961ee08c/attachment.html>


More information about the cfe-commits mailing list