This is the mail archive of the
mailing list for the GNU Fortran project.
Re: [patch, fortran] Fix off-by-one array size in trans-intrinsic.c
- From: Tobias Burnus <burnus at net-b dot de>
- To: FX <fxcoudert at gmail dot com>
- Cc: Fortran List <fortran at gcc dot gnu dot org>, Juha Ruokolainen <Juha dot Ruokolainen at csc dot fi>, gcc-patches at gcc dot gnu dot org
- Date: Wed, 01 Sep 2010 12:29:04 +0200
- Subject: Re: [patch, fortran] Fix off-by-one array size in trans-intrinsic.c
- References: <467F4A46BB08EF409B49ADF9D99FA82D073C0EFA0C@cscexmb01.csc.local> <79631B5A-458E-4203-B26D-61EFD3F5B100@gmail.com>
On 09/01/2010 11:56 AM, FX wrote:
Many thanks to Juha for the bug report, and sorry for not replying earlier. Once I had valgrind installed on my machine, it became clear your issue was reproducible. In fact, I simply made an array that's too small... The attached patch fixes it.
It also removes an unused assignment to real16_decl for complex intrinsics (it was used in a preliminary version of my initial patch, but is not needed anymore).
Bootstrapped and regtested both with TFmode enabled and without, on x86_64-linux. Tested a few testcases under valgrind. OK to commit?
OK. Thanks to Juha for reporting the issue - and thanks FX for this patch.