Re: [Patch] ld: check for address space overflow

[Tristan Gingold <gingold@adacore.com>]

> On 14 Mar 2017, at 13:17, Alan Modra <amodra@gmail.com> wrote:
> 
> On Tue, Mar 14, 2017 at 11:50:27AM +0100, Tristan Gingold wrote:
>> 
>>> On 14 Mar 2017, at 11:19, Alan Modra <amodra@gmail.com> wrote:
>>> The new testcases need some tweaking too.  Even after applying the
>>> above I'm left with:
>>> 
>>> arm-aout  +FAIL: section size overflow
>>> arm-coff  +FAIL: section size overflow
>>> arm-epoc-pe  +FAIL: section size overflow
>>> arm-pe  +FAIL: section size overflow
>>> arm-symbianelf  +FAIL: section size overflow
>>> arm-wince-pe  +FAIL: section size overflow
>>> i386-linuxaout  +FAIL: section size overflow
>>> i386-linuxaout  +FAIL: section size overflow
>>> i386-lynxos  +FAIL: section size overflow
>>> i386-netware  +FAIL: section size overflow
>>> i586-aout  +FAIL: section size overflow
>>> i686-pe  +FAIL: section size overflow
>> 
>> What is the host ?  32 bit or 64 bit ?
> 
> 64 bit.  x86_64-linux.

Would that be ok ?
(No need to bother with additional sections).

Tristan.

commit 6e31e599777fb6a9e09f06b2fc957201b2c7eb4f
Author: Tristan Gingold <gingold@adacore.com>
Date:   Tue Mar 14 14:54:37 2017 +0100

    ld-checks: tweak overflow checks.
    
    	* testsuite/ld-checks/checks.exp (overflow_check): Disable for
    	non-elf targets.

diff --git a/ld/ChangeLog b/ld/ChangeLog
index c164710..c988c3b 100644
--- a/ld/ChangeLog
+++ b/ld/ChangeLog
@@ -1,3 +1,8 @@
+2017-03-14  Tristan Gingold  <gingold@adacore.com>
+
+	* testsuite/ld-checks/checks.exp (overflow_check): Disable for
+	non-elf targets.
+
 2017-03-13  Tristan Gingold  <gingold@adacore.com>
 
 	* ldlang.c (lang_check_section_addresses): Check for address space
diff --git a/ld/testsuite/ld-checks/checks.exp b/ld/testsuite/ld-checks/checks.exp
index 782f50a..e411d03 100644
--- a/ld/testsuite/ld-checks/checks.exp
+++ b/ld/testsuite/ld-checks/checks.exp
@@ -85,6 +85,9 @@ proc overflow_check {} {
 	 && ![istarget arm*-*-*] } {
 	return
     }
+    if ![is_elf_format] {
+	return
+    }
 
     run_dump_test "over"
     run_dump_test "over2"