On 26/09/16 10:15, Richard Biener wrote:
On Thu, 22 Sep 2016, Thomas Preudhomme wrote:
Hi,
This patch is part of a patch series to add support for ARMv8-M[1] to GCC.
This specific patch moves memory model declarations in memmodel.h.
Currently, is_mm_* memory model related functions are declared in tree.h which
prevents using them in constraints machine description files. This patch move
this functions into a new memmodel.h header to allow such a thing and update
all users of this function to include the new header.
Ok. I wonder whether we can move the memmodel stuff from coretypes
as well then?
It's not needed for using what's in there in constraints.md but would be good
for consistency I think. Do you want me to add it to the patch or make a
separate one and commit this one?
Thanks,
Richard.
Best regards,
Thomas
ChangeLog entries are as follows:
*** gcc/ChangeLog ***
2016-04-20 Thomas Preud'homme <thomas.preudho...@arm.com>
* tree.h (memmodel_from_int, memmodel_base, is_mm_relaxed,
is_mm_consume, is_mm_acquire, is_mm_release, is_mm_acq_rel,
is_mm_seq_cst, is_mm_sync): Move to ...
* memmodel.h: This. New file.
* builtins.c: Include memmodel.h.
* optabs.c: Likewise.
* tsan.c: Likewise.
* config/aarch64/aarch64.c: Likewise.
* config/alpha/alpha.c: Likewise.
* config/arm/arm.c: Likewise.
* config/i386/i386.c: Likewise.
* config/ia64/ia64.c: Likewise.
* config/mips/mips.c: Likewise.
* config/rs6000/rs6000.c: Likewise.
* config/sparc/sparc.c: Likewise.
* genconditions.c: Include memmodel.h in generated file.
* genemit.c: Likewise.
* genoutput.c: Likewise.
* genpeep.c: Likewise.
* genpreds.c: Likewise.
* genrecog.c: Likewise.
*** gcc/c-family/ChangeLog ***
2016-04-20 Thomas Preud'homme <thomas.preudho...@arm.com>
* c-common.c: Include memmodel.h.
Is this ok for trunk?
Best regards,
Thomas