On Sun, Aug 21, 2011 at 4:19 PM, David Edelsohn <[email protected]> wrote:
> This patch broke bootstrap on AIX. It emits a ".section" op in
> assembly but ".section" is an ELF syntax op not AIX XCOFF.
>
> FE..initialize_critical:
> .section .init_array
>
> varasm.c should not be generating ELF ops for non-ELF targets.
>
> config.log shows:
>
> gcc_cv_initfini_array=yes
> tm_file_list includes initfini-array.h
> tm_include_list includes initfini-array.h
>
> Why is the patch affecting non-ELF targets? Please fix or revert immediately.
>
I didn't know .init_array section was enabled for AIX. Does this patch
work for you?
Sorry about the breakage.
--
H.J.
diff --git a/gcc/config.gcc b/gcc/config.gcc
index 7f29213..843b382 100644
--- a/gcc/config.gcc
+++ b/gcc/config.gcc
@@ -3058,9 +3058,13 @@ if test x$with_schedule = x; then
esac
fi
-# Support --enable-initfini-array.
+# Support --enable-initfini-array for ELF OSes.
if test x$enable_initfini_array = xyes; then
- tm_file="${tm_file} initfini-array.h"
+ case "${tm_file}" in
+ *elfos.h*)
+ tm_file="${tm_file} initfini-array.h"
+ ;;
+ esac
fi
# Validate and mark as valid any --with options supported
