Re: Starting patch review wiki

Andrew Pinski via Gcc <[email protected]>
Newsgroups gmane.comp.gcc.devel
Message-ID <CALvbMcBPnQzpmkhA+Cc34grLxvFbhhEwnhpdFfuyfYZHt6oy3A@mail.gmail.com>
On Wed, Feb 25, 2026 at 12:39 AM Kyrylo Tkachov via Gcc <[email protected]> wrote:
>
> Hi all,
>
> I gave a talk at last Cauldron in Porto about starting upstream patch review and addressing some common issues that potential new reviewers face. I was asked to put the content together on a wiki page. Apologies for the delay, but I’ve done that now at:
> https://gcc.gnu.org/wiki/StartingPatchReview
> It should be visible from the wiki home page too.
> Feel free to extend it with more review guidance or correct things that you think are wrong.

I added some recommendations on what to recommend for new developers
to run to get either the commit message correct and/or coding style
correct.
Obviously for the forge submissions these 2 are run after granting
approval but for email patch review it would help to recommend them
for new developers.
It should be noted that ./contrib/check_GNU_style.py needs some more
improvements in some areas as it is errors out in some cases that it
should not (I filed 3 bugs yesterday I noticed while reviewing the
ouput from the new backend for wasm) but I think we should still
recommend it.

Thanks,
Andrea


>
> Thanks,
> Kyrill
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.