On Tue, Oct 11, 2016 at 4:39 PM, Pierre-Marie de Rodat <dero...@adacore.com> wrote: > On 09/28/2016 09:48 AM, Richard Biener wrote: >> >> Hmm, interesting approach. It might work reliably at this point of >> the compilation >> but we do actually recycle cgraph nodes. > > > Arf, what a pity. ;-) > >> So I wonder if given we do have >> ->origin / ->next_nested in the cgraph if we can simply perform a walk in >> the >> appropriate order. > > >> But then OTOH in my early LTO debug patchset I have >> […] >> which is not 100% equivalent (looking at the abstract origin) but it >> sounds like a >> similar issue. So I wonder if doing the same for DECL_CONTEXT being a >> function >> makes sense here and would also fix your particular issue in a more >> reliable way. > > > I think it makes sense, and it seems to work, so many thanks for the > suggestion! Bootstrapped and regtested successfuly on x86_64-linux. And this > time, I did not forgot to attach the new testcase!
Ok. Thanks, Richard. > -- > Pierre-Marie de Rodat
