[cfe-commits] [PATCH] More recursive visit removals

Erik Verbruggen erikjv at me.com
Wed Mar 14 08:18:44 PDT 2012


Attached is probably the last one in the series. After that patch, there are two calls to ExprEngine::Visit left. One is in ExprEngine::ProcessStmt, but will have to stay there. The other one is in AggExprVisitor, called from ExprEngine::VisitAggExpr. This last one is only called from disabled code in ExprEngine::VisitCXXConstructExpr. Is it safe to remove this code too?

-- Erik.

-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Remove-unnecessary-recursive-visits-for-ExprWithClea.patch
Type: application/octet-stream
Size: 4056 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20120314/2f13d3b5/attachment.obj>
-------------- next part --------------



On Mar 14, 2012, at 1:21, Ted Kremenek wrote:

> LGTM.
> 
> On Mar 5, 2012, at 4:09 AM, Erik Verbruggen <erikjv at me.com> wrote:
> 
>> This time in the VisitCXXConstructExpr method.
>> 
>> -- Erik.
>> 
>> <0001-Removes-more-recursive-visitations-in-ExprEngine-tha.patch>_______________________________________________
>> 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