Properly check linker warnings

	* lib/ld-lib.exp (run_cc_link_tests): Properly check linker
	warnings.

diff --git a/ld/testsuite/ChangeLog b/ld/testsuite/ChangeLog
index c5126c0..15fb595 100644 (file)
--- a/ld/testsuite/ChangeLog
+++ b/ld/testsuite/ChangeLog
@@ -1,5 +1,10 @@
 2012-12-07  H.J. Lu  <hongjiu.lu@intel.com>
 
+       * lib/ld-lib.exp (run_cc_link_tests): Properly check linker
+       warnings.
+
+2012-12-07  H.J. Lu  <hongjiu.lu@intel.com>
+
        * lib/ld-lib.exp (ar_simple_create): Pass -rc to ar.
 
 2012-12-07  H.J. Lu  <hongjiu.lu@intel.com>

diff --git a/ld/testsuite/lib/ld-lib.exp b/ld/testsuite/lib/ld-lib.exp
index 4ddb659..f08b9bf 100644 (file)
--- a/ld/testsuite/lib/ld-lib.exp
+++ b/ld/testsuite/lib/ld-lib.exp
@@ -1407,21 +1407,26 @@ proc run_cc_link_tests { ldtests } {
            } else {
                set failed 0
            }
-       } elseif { ![ld_simple_link $cc_cmd $binfile "-L$srcdir/$subdir $ldflags $objfiles"] } {
+       } else {
+           if { ![ld_simple_link $cc_cmd $binfile "-L$srcdir/$subdir $ldflags $objfiles"] } {
+               set failed 1
+           } else {
+               set failed 0
+           }
+
            # Check if exec_output is expected.
            if { $warnings != "" } then {
                verbose -log "returned with: <$exec_output>, expected: <$warnings>"
                if { [regexp $warnings $exec_output] } then {
-                   set failed 0
+                   set failed 2
                } else {
                    set failed 1
                }
-           } else {
+           }
+
+           if { $failed == 1 } {
                fail $testname
-               set failed 1
            }
-       } else {
-           set failed 0
        }
 
        if { $failed == 0 } {