[cfe-commits] r143796 - /cfe/trunk/lib/Analysis/LiveVariables.cpp

Ted Kremenek kremenek at apple.com
Sat Nov 5 00:14:12 PDT 2011


This still isn't correct.  We still need the 'while' loop.

On Nov 5, 2011, at 12:09 AM, Argyrios Kyrtzidis wrote:

> Author: akirtzidis
> Date: Sat Nov  5 02:09:40 2011
> New Revision: 143796
> 
> URL: http://llvm.org/viewvc/llvm-project?rev=143796&view=rev
> Log:
> LookThroughStmt GM release.
> 
> 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=143796&r1=143795&r2=143796&view=diff
> ==============================================================================
> --- cfe/trunk/lib/Analysis/LiveVariables.cpp (original)
> +++ cfe/trunk/lib/Analysis/LiveVariables.cpp Sat Nov  5 02:09:40 2011
> @@ -232,10 +232,10 @@
> }
> 
> static const Stmt *LookThroughStmt(const Stmt *S) {
> -  if (const Expr *E = dyn_cast<Expr>(S))
> -    return E->IgnoreParens();
>   if (const OpaqueValueExpr *OVE = dyn_cast<OpaqueValueExpr>(S))
>     return OVE->getSourceExpr()->IgnoreParens();
> +  if (const Expr *E = dyn_cast<Expr>(S))
> +    return E->IgnoreParens();
>   return S;
> }
> 
> 
> 
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits




More information about the cfe-commits mailing list