On Mar 29, 2011, at 1:25 AM, Jay Foad wrote:
> I noticed that:
> 
> - All calls to PHINode::reserveOperandSpace() are on newly created PHINodes.
> - Most places that create a PHINode immediately call
> reserveOperandSpace(), to reserve space for the (known or expected)
> number of operands.
> - Those that don't, probably should.

Good idea!  Thanks for doing this.  The Clang parts look fine to me.

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

Reply via email to