Re: [PATCH] modules: c++: Fix push_namespace ICE with modules

2020-08-27 Thread Nathan Sidwell
On 8/14/20 11:04 AM, Jeff Chapman wrote: Hello! Attached is a patch that fixes an ICE on the devel/c++-modules branch caused by a slot invalidation edge case in push_namespace. I just fell over this myself, reducing a testcase. your fix wasn;t quite right -- we were creating an empty slot in

[PATCH] modules: c++: Fix push_namespace ICE with modules

2020-08-14 Thread Jeff Chapman via Gcc-patches
Hello! Attached is a patch that fixes an ICE on the devel/c++-modules branch caused by a slot invalidation edge case in push_namespace. It's been sitting around for a while and I wasn't sure if I should use the original date or not. Feel free to adjust that to the commit date if that's what it sh