Hi!

On Thu, Aug 17, 2017 at 07:48:54PM -0400, Michael Meissner wrote:
> This patch deletes some of the debug switches that I've added over the years,
> and now don't use any more.
> 
> I did bootstrap builds and make check runs on a little endian power8 system.
> There were no regressions.  Can I check this into the trunk?

Maybe some (or all) of them can be deleted, not leaving a stub?  The
options were undocumented after all, no one should have been using them.

Leaving stubs doesn't hurt much of course, just a bit of clutter.

> +; This option existed in the past, but it hadn't been used in awhile
>  mallow-df-permute
> -Target Undocumented Var(TARGET_ALLOW_DF_PERMUTE) Save
> -; Allow permutation of DF/DI vectors
> +Target RejectNegative Undocumented Ignore

That comment isn't very enlightening, just use the same text as for
the others?

The patch is okay for trunk with or without that change.  Thanks!


Segher

Reply via email to