This is the mail archive of the
fortran@gcc.gnu.org
mailing list for the GNU Fortran project.
Re: [Patch, fortran] PR31879 , PR31197 , PR31258 & PR32703 - various character problems
- From: "Paul Richard Thomas" <paul dot richard dot thomas at gmail dot com>
- To: "Tobias Schlüter" <tobias dot schlueter at physik dot uni-muenchen dot de>, "FX Coudert" <fxcoudert at gmail dot com>
- Cc: "fortran at gcc dot gnu dot org" <fortran at gcc dot gnu dot org>, "gcc-patches List" <gcc-patches at gcc dot gnu dot org>, "Tobias Burnus" <burnus at net-b dot de>
- Date: Thu, 30 Aug 2007 16:28:54 +0200
- Subject: Re: [Patch, fortran] PR31879 , PR31197 , PR31258 & PR32703 - various character problems
- Dkim-signature: a=rsa-sha1; c=relaxed/relaxed; d=gmail.com; s=beta; h=domainkey-signature:received:received:message-id:date:from:to:subject:cc:mime-version:content-type:content-transfer-encoding:content-disposition; b=ea/EhBz45YkW2vsvfkilUAB8o1zHS1TPRSRiqlRK9wL1yS3SWi3ouA3Cn4RXwrdaNyR7+qNsdYHOaU5luzkJOWFpBHnPgk3fg7mhzoHn6c1fjsGEkAoBgIiLgdvHY4Zo2QCaSLKvBb9CL3ULoW2B8tQLE7wF4u03dz5SbeKcsmc=
- Domainkey-signature: a=rsa-sha1; c=nofws; d=gmail.com; s=beta; h=received:message-id:date:from:to:subject:cc:mime-version:content-type:content-transfer-encoding:content-disposition; b=K1+CuMRjOFONb0tZhEx6tatCy47PjD7TEC0E9EEh2wCO67LA7vmtqAy7CrwW9FWUF6XTEt4/KRr4N7fqM6O6wtzxiPqkL47YDHi/tmYFQty7OYXmF++nE1WQYgl33b8Bq2BsoOOfYH3znoup4FDeUW78NUOtUujy+iN/2b1uSQc=
FX and Tobi,
Many thanks for your respective comments:
(i) The hunk in gfc_conv_function_call is indeed unrelated but is only
a format change;
(ii) I will change the initialization of the loop and assert that the
REF_SUBSTRING is the last in the chain - I actually removed this
assert because it seems to be caught further upstream. However, it
canot do any harm;
(iii) The e->ts.type is an error - it is missing == BT_CHARACTER &&
!e->ts.cl. This came about because of bit-rot between the original
writing of the patch and a couple of days ago. I had to apply quite a
lot of the pacth by hand. This caused a regression in CP2K that
Tobias Burnus picked up.
(iv) As for PR32156, looked who fixed it..... Right now, I am not
sure which PR that I meant.
Tobias Burnus and I had an exchange on #gfortran and by email, last
night and this morning. He identified two problems that have been
fixed but need testcases. I will emit the updated patch tonight.
Many thanks to you all
Paul