2008-10-01 H.J. Lu <hongjiu.lu@intel.com>

* binutils-all/group-2.s: New.
	* binutils-all/strip-4.d: Likewise.
	* binutils-all/strip-5.d: Likewise.

	* binutils-all/objcopy.exp: Test objcopy on group-2.s.  Run
	strip-4 and strip-5.

binutils/testsuite/ChangeLog

@@ -1,3 +1,12 @@
+2008-10-01  H.J. Lu  <hongjiu.lu@intel.com>
+
+	* binutils-all/group-2.s: New.
+	* binutils-all/strip-4.d: Likewise.
+	* binutils-all/strip-5.d: Likewise.
+
+	* binutils-all/objcopy.exp: Test objcopy on group-2.s.  Run
+	strip-4 and strip-5.
+
 2008-07-26  Alan Modra  <amodra@bigpond.net.au>
 
 	* binutils-all/objdump.exp: Run compressed debug test only for ELF.

binutils/testsuite/binutils-all/group-2.s

@@ -0,0 +1,8 @@
+	.section .text.foo,"axG",%progbits,.text.foo,comdat
+	.global foo
+foo:
+	.word 0
+	.section .data.bar,"awG",%progbits,.text.foo,comdat
+	.global bar
+bar:
+	.word 0

binutils/testsuite/binutils-all/objcopy.exp

@@ -827,6 +827,7 @@ if { ([istarget "ia64-*-elf*"]
 if [is_elf_format] {
     objcopy_test "ELF unknown section type" unknown.s
     objcopy_test_readelf "ELF group" group.s
+    objcopy_test_readelf "ELF group" group-2.s
     run_dump_test "copy-1"
 }
 
@@ -837,6 +838,8 @@ if [is_elf_format] {
     run_dump_test "strip-1"
     run_dump_test "strip-2"
     run_dump_test "strip-3"
+    run_dump_test "strip-4"
+    run_dump_test "strip-5"
 
     if { [istarget "i*86-*"] || [istarget "x86_64-*-*"] } {
 	# Check to make sure we don't strip a symbol named in relocations.

binutils/testsuite/binutils-all/strip-4.d

@@ -0,0 +1,11 @@
+#PROG: strip
+#source: group-2.s
+#readelf: -Sg --wide
+#name: strip with section group 4
+
+#...
+  \[[ 0-9]+\] \.text.*[ \t]+PROGBITS[ \t0-9a-f]+AX[ \t]+.*
+#...
+  \[[ 0-9]+\] \.data.*[ \t]+PROGBITS[ \t0-9a-f]+WA[ \t]+.*
+#...
+There are no section groups in this file.

binutils/testsuite/binutils-all/strip-5.d

@@ -0,0 +1,18 @@
+#PROG: strip
+#source: group-2.s
+#strip: --strip-unneeded
+#readelf: -Sg --wide
+#name: strip with section group 5
+
+#...
+  \[[ 0-9]+\] .group[ \t]+GROUP[ \t]+.*
+#...
+  \[[ 0-9]+\] \.text.*[ \t]+PROGBITS[ \t0-9a-f]+AXG[ \t]+.*
+#...
+  \[[ 0-9]+\] \.data.*[ \t]+PROGBITS[ \t0-9a-f]+WAG[ \t]+.*
+#...
+COMDAT group section \[[ 0-9]+\] `.group' \[.text.foo\] contains 2 sections:
+   \[Index\]    Name
+   \[[ 0-9]+\]   .text.*
+   \[[ 0-9]+\]   .data.*
+#pass