Patchwork [DOC] Mention -free enabled by default for -O2 and above on AArch64

login
register
mail settings
Submitter Yufeng Zhang
Date March 24, 2014, 5:45 p.m.
Message ID <53306F27.8090304@arm.com>
Download mbox | patch
Permalink /patch/333109/
State New
Headers show

Comments

Yufeng Zhang - March 24, 2014, 5:45 p.m.
Hi,

-free has been enabled by default for -O2 and above on AArch64 a while 
ago.  This patch updates the relevant part of user manual to reflect the 
fact.

OK for stage-4?

Thanks,
Yufeng

gcc/

	* doc/invoke.texi (free): Document AArch64.
Yufeng Zhang - April 11, 2014, 2:46 p.m.
Ping~

Originally posted here:
http://gcc.gnu.org/ml/gcc-patches/2014-03/msg01282.html

Thanks,
Yufeng

On 03/24/14 17:45, Yufeng Zhang wrote:
> Hi,
>
> -free has been enabled by default for -O2 and above on AArch64 a while
> ago.  This patch updates the relevant part of user manual to reflect the
> fact.
>
> OK for stage-4?
>
> Thanks,
> Yufeng
>
> gcc/
>
> 	* doc/invoke.texi (free): Document AArch64.
Gerald Pfeifer - April 13, 2014, 2:30 p.m.
On Fri, 11 Apr 2014, Yufeng Zhang wrote:
> Ping~
> 
> Originally posted here:
> http://gcc.gnu.org/ml/gcc-patches/2014-03/msg01282.html

Looks okay to me, that I'd prefer it to be approved by a target
maintainer -- AArch64 here.

Gerald
Richard Earnshaw - April 14, 2014, 9:36 a.m.
On 13/04/14 15:30, Gerald Pfeifer wrote:
> On Fri, 11 Apr 2014, Yufeng Zhang wrote:
>> Ping~
>>
>> Originally posted here:
>> http://gcc.gnu.org/ml/gcc-patches/2014-03/msg01282.html
> 
> Looks okay to me, that I'd prefer it to be approved by a target
> maintainer -- AArch64 here.
> 
> Gerald
> 

This is fine by me.

R.
Yufeng Zhang - April 14, 2014, 3 p.m.
Thanks.  Patch has been committed to the trunk.  Is it OK for 
gcc-4_9-branch?

Thanks,
Yufeng

On 04/14/14 10:36, Richard Earnshaw wrote:
> On 13/04/14 15:30, Gerald Pfeifer wrote:
>> On Fri, 11 Apr 2014, Yufeng Zhang wrote:
>>> Ping~
>>>
>>> Originally posted here:
>>> http://gcc.gnu.org/ml/gcc-patches/2014-03/msg01282.html
>>
>> Looks okay to me, that I'd prefer it to be approved by a target
>> maintainer -- AArch64 here.
>>
>> Gerald
>>
>
> This is fine by me.
>
> R.
>
Jakub Jelinek - April 14, 2014, 3:02 p.m.
On Mon, Apr 14, 2014 at 04:00:38PM +0100, Yufeng Zhang wrote:
> Thanks.  Patch has been committed to the trunk.  Is it OK for
> gcc-4_9-branch?

Ok.

	Jakub

Patch

diff --git a/gcc/doc/invoke.texi b/gcc/doc/invoke.texi
index 4c183a3..9a41750 100644
--- a/gcc/doc/invoke.texi
+++ b/gcc/doc/invoke.texi
@@ -7414,7 +7414,7 @@  Attempt to remove redundant extension instructions.  This is especially
 helpful for the x86-64 architecture, which implicitly zero-extends in 64-bit
 registers after writing to their lower 32-bit half.
 
-Enabled for x86 at levels @option{-O2}, @option{-O3}.
+Enabled for AArch64 and x86 at levels @option{-O2}, @option{-O3}.
 
 @item -flive-range-shrinkage
 @opindex flive-range-shrinkage