Re: [Patch, fortran, pr65548, addendum] [5/6 Regression] gfc_conv_procedure_call

2015-05-27 Thread Andre Vehreschild
Hi Thomas, thanks for the review. Commited as r223738 with the changes (new testcase, double space in dg-do). Regards, Andre On Wed, 27 May 2015 08:38:07 +0200 Thomas Koenig tkoe...@netcologne.de wrote: Hi Andre, Because this patch is obvious I plan to commit it tomorrow if no one

Re: [Patch, fortran, pr65548, addendum] [5/6 Regression] gfc_conv_procedure_call

2015-05-27 Thread Thomas Koenig
Hi Andre, Because this patch is obvious I plan to commit it tomorrow if no one objects?! The patch itself is obviously OK. About the test case: In general, it is better not to change existing test cases unless absolutely necessary (e.g. adjust an error message). This makes it easier to track

Re: [Patch, fortran, pr65548, addendum] [5/6 Regression] gfc_conv_procedure_call

2015-05-26 Thread Andre Vehreschild
Hi all, unfortunately introduced the latest optimization for this patch an ICE in pFUnit again. The issue occurs when the -fbounds-check is used while allocating an array of non-polymorphic or intrinsic type. It is caused by the location of the gfc_expr of e3rhs not set which is needed to