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: PING: Default to -fstrict-volatile-bitfields for ARM EABI

From: Mark Mitchell <mark at codesourcery dot com>
To: Jie Zhang <jie at codesourcery dot com>
Cc: Richard Kenner <kenner at vlsi1 dot ultra dot nyu dot edu>, rguenther at suse dot de, gcc-patches at gcc dot gnu dot org, nickc at redhat dot com, paul at codesourcery dot com, Richard dot Earnshaw at arm dot com
Date: Mon, 25 Oct 2010 22:28:05 -0400
Subject: Re: PING: Default to -fstrict-volatile-bitfields for ARM EABI
References: <4CC0EA53.4070100@codesourcery.com> <alpine.LNX.2.00.1010221139020.23074@zhemvz.fhfr.qr> <20101025211310.5A62B33C96@vlsi1.gnat.com> <4CC63C41.8070701@codesourcery.com>

On 10/25/2010 10:26 PM, Jie Zhang wrote:

> Not yet. The change has effect on the tests in the patch:
> volatile-bitfields-2.c and volatile-bitfields-3.c, whose records have
> three fields.

I've reviewed the patch (and I know this part of the ARM ABI).  Jie, the
patch is OK.

I'm all for improving the documentation, but you can do that as a
separate patch.  Your change isn't make it any worse.

Thank you,

-- 
Mark Mitchell
CodeSourcery
mark@codesourcery.com
(650) 331-3385 x713

Follow-Ups:
- Re: PING: Default to -fstrict-volatile-bitfields for ARM EABI
  - From: Jie Zhang

References:
- PING: Default to -fstrict-volatile-bitfields for ARM EABI
  - From: Jie Zhang
- Re: PING: Default to -fstrict-volatile-bitfields for ARM EABI
  - From: Richard Guenther
- Re: PING: Default to -fstrict-volatile-bitfields for ARM EABI
  - From: Richard Kenner
- Re: PING: Default to -fstrict-volatile-bitfields for ARM EABI
  - From: Jie Zhang

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