[cfe-commits] r138220 - in /cfe/trunk: include/clang/StaticAnalyzer/Core/PathSensitive/ExprEngine.h lib/StaticAnalyzer/Core/CFRefCount.cpp lib/StaticAnalyzer/Core/ExprEngineCallAndReturn.cpp lib/StaticAnalyzer/Core/ExprEngineObjC.cpp

Ted Kremenek kremenek at apple.com
Sun Aug 21 22:58:53 PDT 2011


On Aug 21, 2011, at 6:03 PM, Jordy Rose <jediknil at belkadan.com> wrote:

> This is running a Debug build and watching the total time of the run, without attempting to quiet the rest of my system -- in other words, entirely anecdotal. I feel like the total time has crept up by a second or so, though.
> 

Ah, you are running the tests.  That wasn't clear.

I'd be very surprised if these changes resulted in a noticeable performance change, even in debug mode.  It's probably worth investigating where the time is going, or actually measuring the time (even 'time make test') to make sure this isn't a real regression.  This really isn't the hot path in the analyzer, so 1 second would be possibly a big deal.

> Also, iterators.cpp takes a LONG time (before and after this change). That might be a good place to look for optimization opportunities?

Agreed.  I've noticed that as well.  It's an experimental checker, so I think the performance issues there are more likely due to the intermediate maturity of that specific checker rather than performance issues with the analyzer core.  Of course some performance analysis should help validate that assumption.



More information about the cfe-commits mailing list