[gfortran] patch for pr libfortran/21303

Jerry DeLisle jvdelisle@verizon.net
Sun Feb 19 20:13:00 GMT 2006


FX Coudert wrote:
>> I am using i686-pc-linux-gnu (aka Fedora Core 4)
> 
> 
> I am also using a i686-pc-linux-gnu. To be sure, I updated my tree, 
> removed other (pending) patches and bootstrapped from scratch. After 
> that, running the fmt_l.f90 testcase gives:
> 
> $ make check-fortran RUNTESTFLAGS="dg.exp=fmt_l.f90"
> [... snip ...]
>                 === gfortran Summary ===
> 
> # of expected passes            152
> /home/fxcoudert/svn/debug/ibin/gcc/testsuite/gfortran/../../gfortran  
> version 4.2.0 20060219 (experimental)
> 
> 
> To be sure, I attached the final patch one again (but it's really the 
> same as the previous diff with the updated fmt_l.f90).
> 
>> At compile time with -std=f95 -pedantic:
>>
>> The warning is:
>>
>> $ gfc -std=f95  fmt_l.f90
>>  In file fmt_l.f90:14
>>
>> or" }
>>     1
>> Warning: Positive width required in format string at (1).  Also notice 
>> the location is off at the end of the comment.
>>
>> So the message does not match the dg-warning.  I gather this is not 
>> what you intended.
> 
> 
> No, the warning I'm expecting happens with -std=gnu -pedantic. I could 
> also test the warning issued with -std=f95, which is as you noted 
> different (because it's the general warning for such problems), but that 
> would mean having either a run-time error (which is harder to test) or 
> no runtime output at all (and I'd like to test the runtime warning).
> 
> So, in short: I'm sure I want to use -std=gnu -pedantic as dg-options, 
> though we still have to figure out why it's working for me and not for 
> you. If someone else could spare a few cycles on this, I'd be glad.
> 
> FX
> 
> 
OK! After a clean bootstrap it passes regression and the test case as expected.

OK to commit to Trunk and 4.1 after the freeze.

Best regards,

Jerry



More information about the Gcc-patches mailing list