On 05/26/2011 09:39 AM, Jason Merrill wrote: > On 05/25/2011 10:21 PM, Nathan Froyd wrote: >> An alternative solution would be to initialize cur_stmt_list somewhere with >> an >> actual 1-element VEC; > > Or just push NULL onto the stack and let append_to_statement_list_1 allocate > the VEC?
Did you misspeak here? append_to_statement_1 shouldn't be caring about VECs. Or do you mean pushing NULL_TREE someplace else, as in: >> the check in add_stmt would then be unnecessary, as we'd >> always be assured of having someplace in the stack to store it. I don't >> trust >> myself to write a patch like that tonight; I'll twiddle with that tomorrow. > > Right, that's what I was thinking about. I think we should only need to do > this once per function. ...here? -Nathan
