Get rid of prefixes in General Dynamic TLS code for x86_64-nacl

gcc/config/i386/i386.md

Index: gcc/config/i386/i386.md
diff --git a/gcc/config/i386/i386.md b/gcc/config/i386/i386.md
index 59c02d088eda0544088f959cae833cd28ad15915..8dc1bdde0c8f1285b4ac072bceec79f67f0c0fe5 100644
--- a/gcc/config/i386/i386.md
+++ b/gcc/config/i386/i386.md
@@ -16450,7 +16450,11 @@
    (unspec:DI [(match_operand:DI 1 "tls_symbolic_operand" "")]
 	      UNSPEC_TLS_GD)]
   "TARGET_64BIT"
-  { return ".byte\t0x66\n\tlea{q}\t{%a1@TLSGD(%%rip), %%rdi|rdi, %a1@TLSGD[rip]}\n" ASM_SHORT "0x6666\n\trex64\n\tcall\t%P2"; }
+{
+  if (TARGET_NACL)
+    return "\tlea{q}\t{%a1@TLSGD(%%rip), %%rdi|rdi, %a1@TLSGD[rip]}\n\tcall\t%P2";
+  return ".byte\t0x66\n\tlea{q}\t{%a1@TLSGD(%%rip), %%rdi|rdi, %a1@TLSGD[rip]}\n" ASM_SHORT "0x6666\n\trex64\n\tcall\t%P2";

[khim, 2011/04/19 15:26:23]

Why not if )...) return ...; else return ... ?

There are significantly more cases where this file uses patters with 

if (check)
  return some_value;
else
  return another_value;

rather then

if (check)
  return some_value;
return another_value;

Like this:

      if (REG_P (operands[2]))
        return "sal{q}\t{%b2, %0|%0, %b2}";
      else if (operands[2] == const1_rtx
               && (TARGET_SHIFT1 || optimize_function_for_size_p (cfun)))
        return "sal{q}\t%0";
      else
        return "sal{q}\t{%2, %0|%0, %2}";

Returns are aligned and this makes the code easier to read and understand...

+}
   [(set_attr "type" "multi")
    (set_attr "length" "16")])