[Patch, Fortran] PR fortran/37193: USE m, ONLY: i, j => i
Tobias Burnus
burnus@net-b.de
Mon Sep 1 07:37:00 GMT 2008
Daniel Kraft wrote:
> Paul Richard Thomas wrote:
>> Your patch looks OK
Ditto from my side.
>> Why does the bit of code in module.c(load_needed)....
>> mio_symbol (sym);
>> sym->attr.use_assoc = 1;
>> if (only_flag)
>> sym->attr.use_only = 1;
>> if (p->u.rsym.renamed)
>> sym->attr.use_rename = 1;
>>
>> return 1;
>>
>> not do the job? Is it needed now, with your patch?
> If you want I can replace the second initialization (the one in your
> snippet) by an assertion and see if the test-suite still passes and if
> it does leave the assertion there.
I think one should do this: For the trunk, such a patch is OK.
For 4.3 I would wait a week or so and add then only the new line without
replacing the old lines with the assert.
Tobias
More information about the Fortran
mailing list