On Tue, 29 Jul 2025, Filip Kastl wrote:
> okay to push? Confirmed that I didn't break anything by running `make
> html`.
...and `make pdf`?
> This patch changes two things. Firstly, we document
> -fdump-rtl-<whatever>-graph and other such options under -fdump-tree.
> At least write a remark about this under -fdump-rtl. Secondly, the
> documentation incorrectly says that -fdump-tree-<whatever>-graph is not
> implemented.
This definitely looks like a nice enhancement; thank you for taking the
time to do so.
> +@option{-fdump-rtl}. This is used for debugging the RTL-based passes of the
> compiler.
This line looks a little long?
> +Here are the possible keywords for @var{pass} and letters for @var{letters},
> +and their meanings:
>
> @table @gcctabopt
Looking at the table, it doesn't actually just list values for @var{pass}
and @var{letters}, if shows fuller options? (That is, it lists "-da"
rather than just "a".)
How about using something like "Here are actual instances of command-line
options following these patterns" (or something different)?
Okay if you take the above into consideration. Happy to look at an updated
patch, alas not required.
Gerald
