Re: [patch, fortran] Fix PR 85631, wrong size checking with allocatable arrays

2018-06-08 Thread Thomas Koenig
Hi Steve, On Fri, Jun 08, 2018 at 09:06:55PM +0200, Thomas Koenig wrote: the attached patch fixes a bug which was uncovered by the PR in a matmul regression. The problem is that bounds checking on the LHS with reallocation on assignment makes no sense, and the original flag was not set for th

Re: [patch, fortran] Fix PR 85631, wrong size checking with allocatable arrays

2018-06-08 Thread Steve Kargl
On Fri, Jun 08, 2018 at 09:06:55PM +0200, Thomas Koenig wrote: > > the attached patch fixes a bug which was uncovered by the PR in > a matmul regression. > > The problem is that bounds checking on the LHS with reallocation on > assignment makes no sense, and the original flag was not set for > th

[patch, fortran] Fix PR 85631, wrong size checking with allocatable arrays

2018-06-08 Thread Thomas Koenig
Hello world, the attached patch fixes a bug which was uncovered by the PR in a matmul regression. The problem is that bounds checking on the LHS with reallocation on assignment makes no sense, and the original flag was not set for the case in question. I added both the original test and the red