Re: [PATCH] Fix GC ICE during simd clone creation (PR middle-end/68339)

2015-11-23 Thread Richard Biener
On Fri, Nov 20, 2015 at 9:03 PM, Jakub Jelinek wrote: > Hi! > > node->get_body () can run various IPA passes and ggc_collect in them Aww. Looks like we never implemented that ggc_defer_collecting idea (don't remember the context this popped up, maybe it was when we introduced

Re: [PATCH] Fix GC ICE during simd clone creation (PR middle-end/68339)

2015-11-23 Thread Jan Hubicka
> On Fri, Nov 20, 2015 at 9:03 PM, Jakub Jelinek wrote: > > Hi! > > > > node->get_body () can run various IPA passes and ggc_collect in them > > Aww. Looks like we never implemented that ggc_defer_collecting idea > (don't remember the context this popped up, maybe it was when

[PATCH] Fix GC ICE during simd clone creation (PR middle-end/68339)

2015-11-20 Thread Jakub Jelinek
Hi! node->get_body () can run various IPA passes and ggc_collect in them, so it is undesirable to hold pointers to GC memory in automatic vars over it. While I could store those vars (clone_info, clone and id) into special GTY vars just to avoid collecting them, it seems easier to call