On Sun, 6 Nov 2016, Uros Bizjak wrote:
> Looks good to me, so OK if tested on non-glibc system.

Thanks, Uros!

Turns out at least in my testing our testsuite is a little non- 
deterministic :-(, but after tests without the patch, with the 
patch, again with the patch, and without the patch again on 
x86_64-unknown-freebsd11.0

   https://gcc.gnu.org/ml/gcc-testresults/2016-12/msg01191.html
   https://gcc.gnu.org/ml/gcc-testresults/2016-12/msg01197.html
   https://gcc.gnu.org/ml/gcc-testresults/2016-12/msg01218.html
   https://gcc.gnu.org/ml/gcc-testresults/2016-12/msg01224.html

I am sufficiently confident none of that noise is related to this patch
and committed it for Roger.  (Thanks again, Roger!)


Uros, okay to also push to the GCC 6 branch for the coming release
and later the GCC 5 branch as well?  For reference, the committed
patch below.

Gerald


2016-12-11  Roger Pau Monné  <roger....@citrix.com>

        * config/i386/x86-64.h: Append --32 to the assembler options when
        -m16 is used on non-glibc systems as well.

Index: config/i386/x86-64.h
===================================================================
--- config/i386/x86-64.h        (revision 243527)
+++ config/i386/x86-64.h        (working copy)
@@ -49,7 +49,7 @@
 #define WCHAR_TYPE_SIZE 32
 
 #undef ASM_SPEC
-#define ASM_SPEC "%{m32:--32} %{m64:--64} %{mx32:--x32}"
+#define ASM_SPEC "%{m16|m32:--32} %{m64:--64} %{mx32:--x32}"
 
 #undef ASM_OUTPUT_ALIGNED_BSS
 #define ASM_OUTPUT_ALIGNED_BSS(FILE, DECL, NAME, SIZE, ALIGN) \

Reply via email to