Re: [PATCH] Fix PR51650

2012-01-03 Thread Richard Guenther
On Mon, 2 Jan 2012, Jason Merrill wrote: > On 01/02/2012 10:49 AM, Richard Guenther wrote: > > For the idea creating the DIE at the point we process the limbo DIE > > yes. But would you consider doing that unconditional or only for LTO? > > Unconditional. Anything that already passed the assert

Re: [PATCH] Fix PR51650

2012-01-02 Thread Jason Merrill
On 01/02/2012 10:49 AM, Richard Guenther wrote: For the idea creating the DIE at the point we process the limbo DIE yes. But would you consider doing that unconditional or only for LTO? Unconditional. Anything that already passed the assert should be unaffected by the change. I can certai

Re: [PATCH] Fix PR51650

2012-01-02 Thread Richard Guenther
On Mon, 2 Jan 2012, Jason Merrill wrote: > It seems like using get_context_die instead of lookup_decl_die might do the > trick. For the idea creating the DIE at the point we process the limbo DIE yes. But would you consider doing that unconditional or only for LTO? I can certainly give it a shot

Re: [PATCH] Fix PR51650

2012-01-02 Thread Jason Merrill
It seems like using get_context_die instead of lookup_decl_die might do the trick. Jason