From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <gcc-patches-return-321480-listarch-gcc-patches=gcc.gnu.org@gcc.gnu.org>
Received: (qmail 24674 invoked by alias); 22 Jun 2012 17:16:07 -0000
Received: (qmail 24597 invoked by uid 22791); 22 Jun 2012 17:16:03 -0000
X-SWARE-Spam-Status: No, hits=-3.3 required=5.0	tests=AWL,BAYES_00,KHOP_RCVD_UNTRUST,KHOP_THREADED,RCVD_IN_DNSWL_LOW,RCVD_IN_HOSTKARMA_NO
X-Spam-Check-By: sourceware.org
Received: from service87.mimecast.com (HELO service87.mimecast.com) (91.220.42.44)    by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Fri, 22 Jun 2012 17:15:51 +0000
Received: from cam-owa1.Emea.Arm.com (fw-tnat.cambridge.arm.com [217.140.96.21]) by service87.mimecast.com; Fri, 22 Jun 2012 18:15:49 +0100
Received: from [10.1.79.66] ([10.1.255.212]) by cam-owa1.Emea.Arm.com with Microsoft SMTPSVC(6.0.3790.0);	 Fri, 22 Jun 2012 18:16:31 +0100
Message-ID: <4FE4A842.8020704@arm.com>
Date: Fri, 22 Jun 2012 17:36:00 -0000
From: Tejas Belagod <tbelagod@arm.com>
User-Agent: Thunderbird 2.0.0.18 (X11/20081120)
MIME-Version: 1.0
To: "gcc-patches@gcc.gnu.org" <gcc-patches@gcc.gnu.org>
CC: Marcus Shawcroft <marcus.shawcroft@arm.com>,  Sofiane Naci <Sofiane.Naci@arm.com>
Subject: Re: [PATCH, GCC][AArch64] Use Enums for TLS dialect option selection
References: <000f01cd3fd4$80a228e0$81e67aa0$@naci@arm.com> <4FD1A915.80004@arm.com>
In-Reply-To: <4FD1A915.80004@arm.com>
X-MC-Unique: 112062218154903101
Content-Type: text/plain; charset=WINDOWS-1252; format=flowed
Content-Transfer-Encoding: quoted-printable
X-IsSubscribed: yes
Mailing-List: contact gcc-patches-help@gcc.gnu.org; run by ezmlm
Precedence: bulk
List-Id: <gcc-patches.gcc.gnu.org>
List-Archive: <http://gcc.gnu.org/ml/gcc-patches/>
List-Post: <mailto:gcc-patches@gcc.gnu.org>
List-Help: <mailto:gcc-patches-help@gcc.gnu.org>
Sender: gcc-patches-owner@gcc.gnu.org
X-SW-Source: 2012-06/txt/msg01485.txt.bz2

Marcus Shawcroft wrote:
> On 01/06/12 09:56, Sofiane Naci wrote:
>> Hi,
>>
>> This patch re-factors TLS dialect option selection in the AArch64 port to
>> use the generic support for enumerated option arguments.
>>
>> Thanks
>> Sofiane
>>
>> -----
>>
>> 2012-06-01  Sofiane Naci<sofiane.naci@arm.com>
>>
>> 	[AArch64] Use Enums for TLS option selection.
>>
>> 	* config/aarch64/aarch64-opts.h (enum aarch64_tls_type): New.
>> 	* config/aarch64/aarch64.c
>> 	(aarch64_tls_dialect): Remove.
>> 	(tls_symbolic_operand_type): Update comment.
>> 	(aarch64_override_options): Remove TLS option setup code.
>> 	* config/aarch64/aarch64.h
>> 	(TARGET_TLS_TRADITIONAL): Remove.
>> 	(TARGET_TLS_DESC): Update definition.
>> 	(enum tls_dialect): Remove.
>> 	(enum tls_dialect aarch64_tls_dialect) Remove.
>> 	* config/aarch64/aarch64.opt
>> 	(tls_type): New.
>> 	(mtls-dialect): Update.
>=20
> OK for aarch64 branch.
> /Marcus
>=20
>=20

I've committed this on the aarch64-branch for Sofiane.

Tejas.