This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: [PATCH 4/4] Set function alignment for M7 to 8 bytes.

From: Sheldon Lobo <sheldon dot lobo at oracle dot com>
To: Eric Botcazou <ebotcazou at adacore dot com>
Cc: gcc-patches at gcc dot gnu dot org
Date: Wed, 17 May 2017 16:36:39 -0400
Subject: Re: [PATCH 4/4] Set function alignment for M7 to 8 bytes.
Authentication-results: sourceware.org; auth=none
References: <1495049778-20262-1-git-send-email-sheldon.lobo@oracle.com> <1495049778-20262-5-git-send-email-sheldon.lobo@oracle.com> <5958779.dQSa2AHR5i@polaris>


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 testsuite directory has its own ChangeLog file so the second item must go
there without the testsuite/ prefix (and "New test" is enough in this case):

	* gcc.target/sparc/niagara7-align.c: New test.


Thanks for the feedback!

Just so I am clear, gcc/testsuite/ChangeLog needs to be
checked into the patch (but gcc/ChangeLog is not)?

Sorry for the newbie question,
Sheldon

Follow-Ups:
- Re: [PATCH 4/4] Set function alignment for M7 to 8 bytes.
  - From: Eric Botcazou

References:
- [PATCH 0/4] Minor SPARC T4 and M7 fixes and additions
  - From: Sheldon Lobo
- [PATCH 4/4] Set function alignment for M7 to 8 bytes.
  - From: Sheldon Lobo
- Re: [PATCH 4/4] Set function alignment for M7 to 8 bytes.
  - From: Eric Botcazou

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]