On 11/04/2011 11:39 AM, Jeff Law wrote:
On 11/03/11 17:49, Andrew MacLeod wrote:
This is the first of 2 parts for the patches to the GCC directory,
it includes the changelog for parts 1 and 2
Primarily it is all the stuff required to create builtins. new
defs, support routines, config changes, etc. It also includes rth's
386 port to the new builtins.
The second part contains the actual built-in code in builtins.c
and optabs.c.
The only comment I've got on this patch is wondering why the -f
options are PARAMS.
I thought PARAMS were used more to tune along a range of values --
recursive depth for inlining, # iterations of certain passes, clamp
number of items on lists, etc.
Unless I'm missing something the new -f options have a range [01] and
could just as easily have been -f<blah> -fno-<blah> options.
Am I missing something?
Long drawn out discussion. It was -f initially, then it was requested
to be changed to -param, and every once in a while, someone asks why it
isn't -f :-)
http://gcc.gnu.org/ml/gcc-patches/2011-07/msg02441.html
Andrew
