This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
RE: [ping] Re: [patch 0/4] reimplement -fstrict-volatile-bitfields, v3
- From: Hans-Peter Nilsson <hp at bitrange dot com>
- To: Bernd Edlinger <bernd dot edlinger at hotmail dot de>
- Cc: Sandra Loosemore <sandra at codesourcery dot com>, GCC Patches <gcc-patches at gcc dot gnu dot org>, Andrew Pinski <pinskia at gmail dot com>, DJ Delorie <dj at redhat dot com>, Richard Biener <richard dot guenther at gmail dot com>, Eric Botcazou <ebotcazou at adacore dot com>
- Date: Tue, 23 Jul 2013 16:37:47 -0400 (EDT)
- Subject: RE: [ping] Re: [patch 0/4] reimplement -fstrict-volatile-bitfields, v3
- References: <51D0F66B dot 6010507 at codesourcery dot com>,<51DC390E dot 9070904 at codesourcery dot com>,<DUB124-W28AB7B318A6EE4701ECED7E4650 at phx dot gbl>,<alpine dot BSF dot 2 dot 02 dot 1307161849040 dot 40754 at arjuna dot pair dot com> <DUB124-W3129A3FA00EC4FE08A9EF0E46E0 at phx dot gbl>
On Tue, 23 Jul 2013, Bernd Edlinger wrote:
> H-P: I hope you can approve my little patch for trunk now,
> although it turned out to be less trivial than I'd have expected.
Sorry, I'm not an approver. (People who are not approvers are
welcome to review any gcc patch where they might say something
useful, whether unique or according to the gcc line, in order to
ease the burden of those with actual approval burdens^wrights.
In this case I *did* ask for the warning, so all the reason for
me to do a little review.)
Thanks! Looks reasonable (to my tree-ignorant eyes) modulo a
few nits:
Please put the "as it would" parts of the changelog entries as
comments in the code instead. (ChangeLog says "what", not "why".)
I'd also tweak the head comment of warn_portable_volatility_p
(like the documentation change) to not refer to
-fstrict-volatile-bitfields as the sole intended cause of
concern; it should instead say something like "at present this
function only covers -fstrict-volatile-bitfields" in order to
open up for future amendments.
Please also change the name to check_portable_volatility instead
of warn_portable_volatility_p; the "_p" suffix is canonically
used for boolean predicates. (You might have copied the wrong
use of _p from somewhere else in the gcc code, but that's also
in error.)
> Of course it is dependent on Sandra's patch part 1 and part 2,
> which must be applied first.
brgds, H-P