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