On 05/26/2011 08:50 PM, Nathan Froyd wrote:
Sigh, I am an idiot. It appears that we always have something pushed by the time add_stmt is called. (I ran into problems implementing the above approach, as I wound up with [ NULL_TREE,<tree> ] and that gave pop_stmt heartburn.) I can't recall why I added the check in the first place; it might have been because I ran into problems in the C FE and blindly assumed I needed the same fix in the C++ FE.
A reasonable assumption; I wonder why C is doing things differently. The patch is OK, but could you look into that inconsistency? Jason