On 11/15/2017 10:38 AM, Wilco Dijkstra wrote:
Sandra Loosemore wrote:

I'd prefer that you remove the reference to configure options entirely
here.  Nowadays most GCC users install a package provided by their OS
distribution, Linaro, etc, rather than trying to build GCC from scratch.

OK, I've removed that reference. Similarly the FRAME_POINTER_REQUIRED
bit as that statement is not only irrelevant but also completely incorrect.

+Enabled at levels @option{-O}, @option{-O1}, @option{-O2}, @option{-O3},
+@option{-Os} and @option{-Og}.

This last sentence makes no sense.  If the option is now enabled by
default, then the optimization level is irrelevant.

It's enabled from -O onwards, so I've changed it to the standard form used
elsewhere and updated the table for -O:

+Enabled by default at @option{-O} and higher.

Here is the cleaned up and simplified version:

[snip]

Thanks, this patch is OK with me.

-Sandra

Reply via email to