We prefer to keep it there. It's all the code that needs to be updated in the near future and was commented out for different reasons.
Thanks, Anna. On Mar 14, 2012, at 4:12 PM, Matt Beaumont-Gay wrote: > Author: matthewbg > Date: Wed Mar 14 18:12:42 2012 > New Revision: 152749 > > URL: http://llvm.org/viewvc/llvm-project?rev=152749&view=rev > Log: > '#if 0' out a variable that's only used in other preprocessor-disabled code. > > (Why are we keeping all of this code around anyway? Say the word and I'll > start swinging the delete hammer.) > > Modified: > cfe/trunk/lib/StaticAnalyzer/Core/ExprEngineCXX.cpp > > Modified: cfe/trunk/lib/StaticAnalyzer/Core/ExprEngineCXX.cpp > URL: > http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/StaticAnalyzer/Core/ExprEngineCXX.cpp?rev=152749&r1=152748&r2=152749&view=diff > ============================================================================== > --- cfe/trunk/lib/StaticAnalyzer/Core/ExprEngineCXX.cpp (original) > +++ cfe/trunk/lib/StaticAnalyzer/Core/ExprEngineCXX.cpp Wed Mar 14 18:12:42 > 2012 > @@ -64,8 +64,10 @@ > ExplodedNode *Pred, > ExplodedNodeSet &destNodes) { > > +#if 0 > const CXXConstructorDecl *CD = E->getConstructor(); > assert(CD); > +#endif > > #if 0 > if (!(CD->doesThisDeclarationHaveABody() && AMgr.shouldInlineCall())) > > > _______________________________________________ > cfe-commits mailing list > [email protected] > http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits _______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
