On Fri, Jun 1, 2012 at 3:09 PM, Richard Trieu <[email protected]> wrote:

> On Wed, May 30, 2012 at 5:37 PM, Chandler Carruth <[email protected]>wrote:
>
>> On Fri, May 18, 2012 at 3:35 PM, Richard Trieu <[email protected]> wrote:
>>
>>> Update the uninitialized field checker to catch more cases of
>>> uninitialized use.  This will catch fields passed by value to function
>>> calls and properly parse conditional operators.  Instead of a recursive
>>> function, an EvaluatedExprVisitor will be used.
>>>
>>
>> What's the compile-time impact of this? Maybe synthesize a test case with
>> a crazy number of fields (all initialized) with long expressions in their
>> initializers, and compare before and after? (There are some tests in INPUTS
>> that you can model this after...)
>>
>
> Synthesized some test cases with 4096 classes, each class having a single
> field initializer with 2048 sub expressions.  My testing showed a 5-10%
> slow-down for the new EvaluatedExprVisitor compared to the current
> recursive function check.
>

Ping.
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to