[llvm-commits] [cfe-commits] [PATCH] remove PHINode::reserveOperandSpace()

John McCall rjmccall at apple.com
Tue Mar 29 10:11:07 PDT 2011


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.



More information about the llvm-commits mailing list