Please excuse some cheap points:

On Mon, 23 Mar 2020, kamlesh kumar via Gcc-patches wrote:

> Attached patch fixes.
> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88877.
> ChangeLog Entry.
>
> 2020-03-23  Kamlesh Kumar  <kamleshbha...@gmail.com>
>
>         * rtl.h : Defined Tuple for bundling rtx, mode and
> unsignedness default as 0
>         Added Extra argument (unsignedp) in emit_library_call and
> emit_library_call_value.

Formatting issue: "." ends sentences, but you should not
terminate the line.  No space before ":".  See other entries.

>         * except.c : Likewise.
>         * explow.c : Likewise.
>         * expmed.c : Likewise.
>         * expr.c : Likewise.
>         * optabs.c : Likewise.
>         * asan.c : Likewise.
>         * builtins.c : Likewise.
>         * calls.c : Likewise.
>         * cfgexpand.c : Likewise.
>         * config/aarch64/aarch64.c : Likewise.
>         * config/aarch64/aarch64.h : Likewise.
>         * config/aarch64/atomics.md : Likewise.
>         * config/alpha/alpha.c : Likewise.
>         * config/arc/arc.c : Likewise.

...

The new argument is default 0, so make use of that: do not patch
all those files where 0 is fine.

brgds, H-P

Reply via email to