http://gcc.gnu.org/bugzilla/show_bug.cgi?id=48426

--- Comment #10 from Steve Kargl <sgk at troutmask dot apl.washington.edu> 
2011-04-15 16:02:17 UTC ---
On Fri, Apr 15, 2011 at 03:29:36PM +0000, jvdelisle at gcc dot gnu.org wrote:
> 
> I have looked over the patch.  I need to do some tests and I have about three
> bugs in front of this one so please give me a few days.
> 

I skimmed the patch.  It is still wrong.

These parts should not be needed:

+    if (!saw_r8)
+      fatal_error ("real kind=8 not available for -freal-4-real-8 option");

+    if (!saw_r4)
+      fatal_error ("real kind=4 not available for -freal-8-real-4 option");

gfortran always has real(4) and real(8) on all important targets.

I also think that the description is the manual is lacking.  For example,

+freal-4-real-8
+Fortran RejectNegative
+Interpret any 4-byte real as an 8-byte real

I think the word 'any' is incorrect.

There also should be a warning here.  "Warning: This option is provided
for debugging purposes, and may give unexpected results for general
use."

Reply via email to