[patch][gold] Fix case where a empty section defines the segment of a following section

Ian Lance Taylor iant@google.com
Wed May 19 03:52:00 GMT 2010


Rafael Espindola <espindola@google.com> writes:

> The updated patch attached runs readelf to check that .init and .text
> are in the same segment.


> --- /dev/null
> +++ b/gold/testsuite/script_test_9.cc
> @@ -0,0 +1,29 @@
> +// script_test_9.cc -- a test case for gold
> +
> +// Copyright 2009 Free Software Foundation, Inc.
> +// Written by Cary Coutant <ccoutant@google.com>.

Fix copyright year and author.


> diff --git a/gold/testsuite/script_test_9.sh b/gold/testsuite/script_test_9.sh
> new file mode 100755
> index 0000000..9f2b7ce
> --- /dev/null
> +++ b/gold/testsuite/script_test_9.sh
> @@ -0,0 +1,42 @@
> +#!/bin/sh
> +
> +# script_test_8.sh -- Check that the script_test_9.t script has placed
> +# .init and .text in the same segment.
> +
> +# Copyright 2009 Free Software Foundation, Inc.
> +# Written by Doug Kwan <dougkwan@google.com>.

Fix file name, copyright year, and author.


> +check script_test_9.stdout "LOAD .*R E "
> +check script_test_9.stdout "LOAD .*RW "
> +check script_test_9.stdout "00     .text .init"
> +check script_test_9.stdout "01     .data "

The last two tests should probably be more flexible about white
space.


This is OK with those changes.

Thanks.

Ian



More information about the Binutils mailing list