Re: [PATCH] Fix cp_binding_level reuse logic

2016-01-28 Thread Jason Merrill
On 01/27/2016 07:44 PM, Patrick Palka wrote: In begin_scope(), we are accidentally clearing the entire free_binding_level store whenever we reuse a single GC-alloced cp_binding_level structure from it. This happens because we erroneously update free_binding_level _after_ the pointer pointing to

[PATCH] Fix cp_binding_level reuse logic

2016-01-27 Thread Patrick Palka
In begin_scope(), we are accidentally clearing the entire free_binding_level store whenever we reuse a single GC-alloced cp_binding_level structure from it. This happens because we erroneously update free_binding_level _after_ the pointer pointing to the next available structure has been cleared,