Hi Iain, > This patch removes the monolithic and cumbersome to maintain > core/threadasm.S, and splitted it into multiple parts, one for each > intended target cpu/os. > > Added .type and .size directives for all asm implementations of > fiber_switchContent and callWithStackShell where they were missing, > fixing the last remaining failing test on -m32 that I can reproduce > locally. > > Bootstrapped and regression tested on x86_64-linux-gnu with multilibs > -m32, -mx32, and -m64. > > Committed to trunk as r270560.
this patch badly broke x86 libphobos testing, as noticed during a i386-pc-solaris2.11 build. All link tests FAIL like this: FAIL: libphobos.cycles/mod1.d --DRT-oncycle=print (test for excess errors) Excess errors: Undefined first referenced symbol in file fiber_switchContext /var/gcc/gcc-9.0.1-20190425/11.5-gcc-gas-libphobos/i386-pc-solaris2.11/libphobos/libdruntime/.libs/libgdruntime.a(thread.o) ld: fatal: symbol referencing errors collect2: error: ld returned 1 exit status UNRESOLVED: libphobos.cycles/mod1.d --DRT-oncycle=print compilation failed to produce executable You need to quote brackets to get them from m4/autoconf macros into configure. Fixed as follows, tested on i386-pc-solaris2.11, installed as obvious. Rainer -- ----------------------------------------------------------------------------- Rainer Orth, Center for Biotechnology, Bielefeld University 2019-04-25 Rainer Orth <r...@cebitec.uni-bielefeld.de> * m4/druntime/cpu.m4 (DRUNTIME_CPU_SOURCES): Quote brackets. * configure: Regenerate.
# HG changeset patch # Parent a7a44865fd130abcbd002e700515b1b113704d0c Fix typo in m4/druntime/cpu.m4 diff --git a/libphobos/configure b/libphobos/configure --- a/libphobos/configure +++ b/libphobos/configure @@ -14050,7 +14050,7 @@ fi powerpc64) druntime_target_cpu_parsed="powerpc64" ;; - i3456786|x86_64) + i[34567]86|x86_64) druntime_target_cpu_parsed="x86" ;; esac diff --git a/libphobos/m4/druntime/cpu.m4 b/libphobos/m4/druntime/cpu.m4 --- a/libphobos/m4/druntime/cpu.m4 +++ b/libphobos/m4/druntime/cpu.m4 @@ -22,7 +22,7 @@ AC_DEFUN([DRUNTIME_CPU_SOURCES], powerpc64) druntime_target_cpu_parsed="powerpc64" ;; - i[34567]86|x86_64) + i[[34567]]86|x86_64) druntime_target_cpu_parsed="x86" ;; esac