Hi Guys,

  The new MSP430 target has a few limitations which show up as
  unexpected failures in the GCC testsuite.  I would like to apply the
  patch below to update target-supports.exp with this information.
  Applying the patch results in a drop of 133 unexpected failures (per
  multilib).

  OK to apply ?

Cheers
  Nick

gcc/testsuite/ChangeLog
2013-09-16  Nick Clifton  <ni...@redhat.com>

        * lib/target-supports.exp (check_effective_target_trampolines):
        Add MSP430 to the list of targets that do not support
        trampolines.
        (check_profiling_available): Add MSP430 to the list of targets
        that do not support profiling.
        (check_effective_target_tls_runtime): Add MSP430 to the list of
        targets that do not support TLS.

Index: gcc/testsuite/lib/target-supports.exp
===================================================================
--- gcc/testsuite/lib/target-supports.exp       (revision 202612)
+++ gcc/testsuite/lib/target-supports.exp       (working copy)
@@ -435,9 +435,10 @@
       return 0
     }
     if { [istarget avr-*-*]
+        || [istarget msp430-*-*]
         || [istarget hppa2.0w-hp-hpux11.23]
-       || [istarget hppa64-hp-hpux11.23] } {
-       return 0;   
+        || [istarget hppa64-hp-hpux11.23] } {
+       return 0;
     }
     return 1
 }
@@ -535,6 +536,7 @@
             || [istarget mmix-*-*]
             || [istarget mn10300-*-elf*]
             || [istarget moxie-*-elf*]
+            || [istarget msp430-*-*]
             || [istarget picochip-*-*]
             || [istarget powerpc-*-eabi*]
             || [istarget powerpc-*-elf]
@@ -652,6 +654,11 @@
 # Return 1 if TLS executables can run correctly, 0 otherwise.
 
 proc check_effective_target_tls_runtime {} {
+    # MSP430 runtime does not have TLS support, but just
+    # running the test below is insufficient to show this.
+    if { [istarget msp430-*-*] } {
+       return 0
+    }
     return [check_runtime tls_runtime {
        __thread int thr = 0;
        int main (void) { return thr; }

Reply via email to