This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH, 1/7] Move memory model declarations in memmodel.h
- From: Richard Biener <rguenther at suse dot de>
- To: Thomas Preudhomme <thomas dot preudhomme at foss dot arm dot com>
- Cc: "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>, Jakub Jelinek <jakub at redhat dot com>
- Date: Mon, 26 Sep 2016 11:15:17 +0200 (CEST)
- Subject: Re: [PATCH, 1/7] Move memory model declarations in memmodel.h
- Authentication-results: sourceware.org; auth=none
- References: <5c3a83ab-454c-b3e0-3683-5d7bf53a3093@foss.arm.com>
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?
Thanks,
Richard.
> ChangeLog entries are as follows:
>
> *** gcc/ChangeLog ***
>
> 2016-04-20 Thomas Preud'homme <thomas.preudhomme@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.preudhomme@arm.com>
>
> * c-common.c: Include memmodel.h.
>
>
> Is this ok for trunk?
>
> Best regards,
>
> Thomas
>
--
Richard Biener <rguenther@suse.de>
SUSE LINUX GmbH, GF: Felix Imendoerffer, Jane Smithard, Graham Norton, HRB 21284 (AG Nuernberg)