This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] arc backend patches. Mainline.
- From: "Giovanni Bajo" <giovannibajo at libero dot it>
- To: "Ramana Radhakrishnan" <ramana dot radhakrishnan at codito dot com>
- Cc: <gcc-patches at gcc dot gnu dot org>
- Date: Mon, 18 Oct 2004 13:30:18 +0200
- Subject: Re: [PATCH] arc backend patches. Mainline.
- References: <41736748.50006@codito.com>
Ramana Radhakrishnan wrote:
> These are the patches for PRs 17317 and 17889 applicable on mainline
> as well. We had first observed these on the 3.4 branch. However these
> are visible also on the mainline.
As a matter of fact, you should always post patches for mainline first. Our
policy says that we always fix bugs on mainline and then, if and only if the
bug can be proven to be a regression, we (try to) backport them to the release
branches.
Are these bugs regressions? A regression is defined as a bug which appeared in
a later version of GCC, and was not present before. Usually we check up to GCC
2.95.
> Ran the testsuite for mainline 20041510.
You must mention which target you built. I assume arc-elf? Was it a cross? On
which host?
> Ok to commit ?
I cannot approve it, but the ChangeLog is wrongly formatted. Please, see a
ChangeLog entry for any other patch and strictly copy the formatting including:
- The way the date is written (YYYY-MM-DD including the hypens)
- The number of spaces between date and name and name and e-mails.
- They way the entry is formatted, how the file names ad function names are
indicated.
- The single tabulation of indent.
- The way the PR number is written (no # sign)
Also, do not merge the patch together. Please send the patches separately like
you did for 3.4. You can still send them in a single mail, but then attacch
different files, each one which a patch and its ChangeLog.
Thanks,
Giovanni Bajo