> The gcc source tree is as desired, but I may have made
> 2 inadvertent process mistakes:
> * did not send a revised PATCH to gcc-patches
That's OK since only a minor correction was requested.
> * made the 4 changes with 1 commit
That's as expected.
--
Eric Botcazou
On 05/19/2017 02:45 PM, Eric Botcazou wrote:
This 4/4 commit needs to contain gcc/config/sparc/sparc.c, gcc/ChangeLog,
gcc/testsuite/gcc.target/sparc/niagara7-align.c and gcc/testsuite/ChangeLog.
The rule is that you put the ChangeLog entry into the ChangeLog file of the
directory where the ch
> The rule is that you put the ChangeLog entry into the ChangeLog file of the
> directory where the change is made, or that of the parent directory if there
> is none, recursively. The filename in that ChangeLog file must be relative
> to the directory when the ChangeLog file is.
...where...
--
> Just so I am clear, gcc/testsuite/ChangeLog needs to be
> checked into the patch (but gcc/ChangeLog is not)?
This 4/4 commit needs to contain gcc/config/sparc/sparc.c, gcc/ChangeLog,
gcc/testsuite/gcc.target/sparc/niagara7-align.c and gcc/testsuite/ChangeLog.
The rule is that you put the Chang
On 05/17/2017 04:11 PM, Eric Botcazou wrote:
* config/sparc/sparc.c (sparc_option_override): Set function
alignment for -mcpu=niagara7 to 64 to match the I$ line.
* testsuite/gcc.target/sparc/niagara7-align.c: Test case with
-mcpu=niagara7 -falign-functions.
The
> * config/sparc/sparc.c (sparc_option_override): Set function
> alignment for -mcpu=niagara7 to 64 to match the I$ line.
> * testsuite/gcc.target/sparc/niagara7-align.c: Test case with
> -mcpu=niagara7 -falign-functions.
The testsuite directory has its own ChangeLog file s
* config/sparc/sparc.c (sparc_option_override): Set function
alignment for -mcpu=niagara7 to 64 to match the I$ line.
* testsuite/gcc.target/sparc/niagara7-align.c: Test case with
-mcpu=niagara7 -falign-functions.
---
gcc/config/sparc/sparc.c
