Re: pr59016

2015-04-08 Thread Mikael Morin
Le 08/04/2015 12:29, Mikael Morin a écrit : > Except that the following ... ahem ... doesn't work. > And it doesn't work because gfc_get_ha_symbol doesn't version host-associated symbols. So one has to call symbol.c's save_symbol_data by hand. And then, we can as well keep the original gfc_find_s

Re: pr59016

2015-04-08 Thread Mikael Morin
Le 07/04/2015 14:25, Mikael Morin a écrit : > Le 06/04/2015 20:26, Mikael Morin a écrit : >> Regarding the patch, I don't understand why the existing symbol >> restoration code doesn't work here (see >> gfc_restore_last_undo_checkpoint, restore_old_symbol). I have to >> investigate more. > > I th

Re: pr59016

2015-04-07 Thread Mikael Morin
Le 06/04/2015 20:26, Mikael Morin a écrit : > Regarding the patch, I don't understand why the existing symbol > restoration code doesn't work here (see > gfc_restore_last_undo_checkpoint, restore_old_symbol). I have to > investigate more. I think the problem is the usage of gfc_find_symbol in gfc

Re: pr59016

2015-04-07 Thread Dominique Dhumieres
Dear Evangelos, I have posted at https://gcc.gnu.org/bugzilla/attachment.cgi?id=35240 a reformatted patch along the Mikael's comment. I have also fixed a failure in the test (parentheses have to be escaped). The patch fixes not only pr59016, but also pr50259 (there are probably random ICEs?) pr56

Re: pr59016

2015-04-06 Thread Mikael Morin
Le 06/04/2015 01:04, Evangelos Drikos a écrit : > > Hi, > > The attached patch, type 0, has been discussed a little at: > https://gcc.gnu.org/bugzilla/show_bug.cgi?id=59016 > > Yet, the final version submitted is slightly different from the ones > discussed and tested in the above link. > > Ha