Re: [PATCH] tell gcc optimizer to never introduce new data races

From: Peter Zijlstra
Date: Tue Jun 10 2014 - 11:13:51 EST


On Tue, Jun 10, 2014 at 05:04:55PM +0200, Marek Polacek wrote:
> On Tue, Jun 10, 2014 at 04:53:27PM +0200, Peter Zijlstra wrote:
> > On Tue, Jun 10, 2014 at 03:23:36PM +0200, Jiri Kosina wrote:
> > > +# Tell gcc to never replace conditional load with a non-conditional one
> > > +KBUILD_CFLAGS += $(call cc-option,--param allow-store-data-races=0)
> > > +
> >
> > Why do we not want: -fmemory-model=safe? And should we not at the very
> > least also disable packed-store-data-races?
>
> Note that the option does not exist, even though it is mentioned in the
> documentation.

Urgh.. ok. Any word on the packed-store-data thing?

Attachment: pgp8cLy88nTmo.pgp
Description: PGP signature