This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]

Re: a language-specific options hook


>I was reminded today about a piece of uglyness I'd been intending
>to clean up for some time.  The fact that I no longer have to add
>-fset-g77-defaults to the f771 gdb command line is merely coincidental.

Looks pretty good, except you left out one line:

- 	  ffe_is_do_internal_checks_ = 0;

You might want to put that back in the lang_init code for g77, if
you have a chance.

Or, you could just change the initial value of it, in top.c (IIRC).

The idea of that global is to inhibit moderately expensive start-up
checks g77 might perform, such as verifying that tables of intrinsics
are in the right order, have coherent signature specifications, etc.

I'll probably modify egcs-g77 later so that specifying -fversion
turns on this flag, so anyone doing `g77 -v' will run the checks.
(`g77 -v' behavior of official g77 will be restored by me in
egcs-g77 within a week or so, maybe even in time for the next
snapshot.)

So, if you feel truly motivated (I'd do it if I had CVS access) and
wanted to change the global init from `= TRUE;' to `= FALSE;', and
the handling of

      if (strcmp (&opt[2], "version") == 0)
	ffe_set_is_version (TRUE);

to

      if (strcmp (&opt[2], "version") == 0)
	{
	  ffe_set_is_version (TRUE);
	  ffe_is_do_internal_checks_ = TRUE;
	}

then by all means feel free!  :)

        tq vm, (burley)


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]