Re: [PATCH v2] c++, coroutines: Split the ramp build into a separate function.

On 8/22/24 7:29 AM, Iain Sandoe wrote: + tree fn_return_type = TREE_TYPE (TREE_TYPE (orig)); /* Ramp: */ + tree stmt = begin_function_body (); The name "stmt" doesn't suggest to me that it's holding the result of begin_function_body. Maybe "ramp_fnbody"? Of course, then there's some

[PATCH v2] c++, coroutines: Split the ramp build into a separate function.

>>+ tree fn_return_type = TREE_TYPE (TREE_TYPE (orig)); >> /* Ramp: */ >>+ tree stmt = begin_function_body (); >The name "stmt" doesn't suggest to me that it's holding the result of >begin_function_body. Maybe "ramp_fnbody"? Of course, then there's some >confusion with "ramp_body". Shou