public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
From: Jiong Wang <jiong.wang@arm.com>
To: GCC Patches <gcc-patches@gcc.gnu.org>
Cc: Marcus Shawcroft <marcus.shawcroft@gmail.com>
Subject: [AArch64][TLSLE][2/3] Add the option "-mtls-size" for AArch64
Date: Wed, 19 Aug 2015 14:37:00 -0000	[thread overview]
Message-ID: <n99h9nv5p21.fsf@arm.com> (raw)
In-Reply-To: <n99io8b5p6s.fsf@arm.com>

[-- Attachment #1: Type: text/plain, Size: 710 bytes --]


As we have added -mtls-size support, there should be four types TLSLE
symbols:

  SYMBOL_TLSLE12
  SYMBOL_TLSLE24
  SYMBOL_TLSLE32
  SYMBOL_TLSLE48

which reflect the maximum address bits needed to address this symbol.

This patch rename SYMBOL_TLSLE to SYMBOL_TLSLE24. Patch [3/3] will add
support for other symbol types.

OK for trunk?

2015-08-19  Jiong Wang  <jiong.wang@arm.com>

gcc/
  * config/aarch64/aarch64-protos.h (aarch64_symbol_type): Rename
  SYMBOL_TLSLE to SYMBOL_TLSLE24.
  * config/aarch64/aarch64.c (aarch64_load_symref_appropriately): Likewise
  (aarch64_expand_mov_immediate): Likewise
  (aarch64_print_operand): Likewise
  (aarch64_classify_symbol): Likewise


[-- Attachment #2: 0002-2.patch --]
[-- Type: text/x-diff, Size: 2727 bytes --]

From 676fc22d51432b037a2c77ae9de01f934cc77985 Mon Sep 17 00:00:00 2001
From: Jiong Wang <jiong.wang@arm.com>
Date: Wed, 19 Aug 2015 14:12:57 +0100
Subject: [PATCH 2/3] 2

---
 gcc/config/aarch64/aarch64-protos.h |  4 ++--
 gcc/config/aarch64/aarch64.c        | 12 ++++++------
 2 files changed, 8 insertions(+), 8 deletions(-)

diff --git a/gcc/config/aarch64/aarch64-protos.h b/gcc/config/aarch64/aarch64-protos.h
index 0b09d49..daa45bf 100644
--- a/gcc/config/aarch64/aarch64-protos.h
+++ b/gcc/config/aarch64/aarch64-protos.h
@@ -74,7 +74,7 @@ enum aarch64_symbol_context
    SYMBOL_SMALL_TLSGD
    SYMBOL_SMALL_TLSDESC
    SYMBOL_SMALL_GOTTPREL
-   SYMBOL_TLSLE
+   SYMBOL_TLSLE24
    Each of these represents a thread-local symbol, and corresponds to the
    thread local storage relocation operator for the symbol being referred to.
 
@@ -111,7 +111,7 @@ enum aarch64_symbol_type
   SYMBOL_SMALL_GOTTPREL,
   SYMBOL_TINY_ABSOLUTE,
   SYMBOL_TINY_GOT,
-  SYMBOL_TLSLE,
+  SYMBOL_TLSLE24,
   SYMBOL_FORCE_TO_MEM
 };
 
diff --git a/gcc/config/aarch64/aarch64.c b/gcc/config/aarch64/aarch64.c
index f55cc38..87f8d96 100644
--- a/gcc/config/aarch64/aarch64.c
+++ b/gcc/config/aarch64/aarch64.c
@@ -1115,7 +1115,7 @@ aarch64_load_symref_appropriately (rtx dest, rtx imm,
 	return;
       }
 
-    case SYMBOL_TLSLE:
+    case SYMBOL_TLSLE24:
       {
 	rtx tp = aarch64_load_tp (NULL);
 
@@ -1677,7 +1677,7 @@ aarch64_expand_mov_immediate (rtx dest, rtx imm)
 
 	case SYMBOL_SMALL_ABSOLUTE:
 	case SYMBOL_TINY_ABSOLUTE:
-	case SYMBOL_TLSLE:
+	case SYMBOL_TLSLE24:
 	  aarch64_load_symref_appropriately (dest, imm, sty);
 	  return;
 
@@ -4560,7 +4560,7 @@ aarch64_print_operand (FILE *f, rtx x, char code)
 	  asm_fprintf (asm_out_file, ":gottprel:");
 	  break;
 
-	case SYMBOL_TLSLE:
+	case SYMBOL_TLSLE24:
 	  asm_fprintf (asm_out_file, ":tprel:");
 	  break;
 
@@ -4593,7 +4593,7 @@ aarch64_print_operand (FILE *f, rtx x, char code)
 	  asm_fprintf (asm_out_file, ":gottprel_lo12:");
 	  break;
 
-	case SYMBOL_TLSLE:
+	case SYMBOL_TLSLE24:
 	  asm_fprintf (asm_out_file, ":tprel_lo12_nc:");
 	  break;
 
@@ -4611,7 +4611,7 @@ aarch64_print_operand (FILE *f, rtx x, char code)
 
       switch (aarch64_classify_symbolic_expression (x, SYMBOL_CONTEXT_ADR))
 	{
-	case SYMBOL_TLSLE:
+	case SYMBOL_TLSLE24:
 	  asm_fprintf (asm_out_file, ":tprel_hi12:");
 	  break;
 	default:
@@ -8717,7 +8717,7 @@ aarch64_classify_tls_symbol (rtx x)
       return SYMBOL_SMALL_GOTTPREL;
 
     case TLS_MODEL_LOCAL_EXEC:
-      return SYMBOL_TLSLE;
+      return SYMBOL_TLSLE24;
 
     case TLS_MODEL_EMULATED:
     case TLS_MODEL_NONE:
-- 
1.9.1


  reply	other threads:[~2015-08-19 14:30 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-05-21 16:53 [AArch64][TLSLE][5/N] Recognize -mtls-size Jiong Wang
2015-05-21 18:14 ` [AArch64][TLSLE][4/N] " Jiong Wang
2015-06-26 15:25 ` [AArch64][TLSLE][5/N] " Marcus Shawcroft
2015-08-19 14:30   ` [AArch64][TLSLE][1/3] Add the option "-mtls-size" for AArch64 Jiong Wang
2015-08-19 14:37     ` Jiong Wang [this message]
2015-08-19 16:34       ` [AArch64][TLSLE][2/3] Rename SYMBOL_TLSLE to SYMBOL_TLSLE24 Jiong Wang
2015-08-25  9:46       ` [AArch64][TLSLE][2/3] Add the option "-mtls-size" for AArch64 Marcus Shawcroft
2015-08-25 10:26     ` [AArch64][TLSLE][1/3] " Marcus Shawcroft
2015-08-25 14:19       ` Jiong Wang
2015-08-26 12:39         ` Marcus Shawcroft

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=n99h9nv5p21.fsf@arm.com \
    --to=jiong.wang@arm.com \
    --cc=gcc-patches@gcc.gnu.org \
    --cc=marcus.shawcroft@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).