[PATCH, GCC][AArch64] Use Enums for code models option selection

[Sofiane Naci]

Hi,

I discovered a bug in my previous patch, so I attach a new one.
The ChangeLog hasn't changed.
OK to commit?

Thanks
Sofiane

> -----Original Message-----
> From: gcc-patches-owner@gcc.gnu.org [mailto:gcc-patches-owner@gcc.gnu.org]
On
> Behalf Of Sofiane Naci
> Sent: 31 May 2012 10:55
> To: gcc-patches@gcc.gnu.org
> Subject: [PATCH, GCC][AArch64] Use Enums for code models option selection
> 
> Hi,
> 
> This patch re-factors code models option selection in the AArch64 port:
> 
>  . Renaming variables such as mem_model to cmodel, for better clarity.
>  . Using the generic support for enumerated option arguments.
>  . Fixing touched code layout and formatting issues.
> 
> Thanks
> Sofiane
> 
> -----
> 
> ChangeLog:
> 
> 2012-05-31  Sofiane Naci <sofiane.naci@arm.com>
> 
> 	[AArch64] Use Enums for code models option selection.
> 
> 	* config/aarch64/aarch64-elf-raw.h (AARCH64_DEFAULT_MEM_MODEL):
> Delete.
> 	* config/aarch64/aarch64-linux.h (AARCH64_DEFAULT_MEM_MODEL):
> Delete.
> 	* config/aarch64/aarch64-opts.h (enum aarch64_code_model): New.
> 	* config/aarch64/aarch64-protos.h: Update comments.
> 	* config/aarch64/aarch64.c: Update comments.
> 	(aarch64_default_mem_model): Rename to aarch64_code_model.
> 	(aarch64_expand_mov_immediate): Remove error message.
> 	(aarch64_select_rtx_section): Remove assertion and update comment.
> 	(aarch64_override_options): Move memory model initialization from
> here.
> 	(struct aarch64_mem_model): Delete.
> 	(aarch64_memory_models[]): Delete.
> 	(initialize_aarch64_memory_model): Rename to
> initialize_aarch64_code_model
> 	and update.
> 	(aarch64_classify_symbol): Handle AARCH64_CMODEL_TINY and
> 	AARCH64_CMODEL_TINY_PIC
> 	* config/aarch64/aarch64.h
> 	(enum aarch64_memory_model): Delete.
> 	(aarch64_default_mem_model): Rename to aarch64_cmodel.
> 	(HAS_LONG_COND_BRANCH): Update.
> 	(HAS_LONG_UNCOND_BRANCH): Update.
> 	* config/aarch64/aarch64.opt
> 	(cmodel): New.
> 	(mcmodel): Update.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: aarch64-use-enums-for-cmodels.patch
Type: application/octet-stream
Size: 10964 bytes
Desc: not available
URL: <http://gcc.gnu.org/pipermail/gcc-patches/attachments/20120613/81298586/attachment.obj>