On 03/12/2018 06:31 PM, Jakub Jelinek wrote: > On Mon, Mar 12, 2018 at 06:28:09PM +0100, Andreas Krebbel wrote: >> Currently the default ABI option for a GCC built on a 64 bit system is >> always -m64. This is inconvenient when e.g. building 32 bit libraries >> on a 64 bit system. The usual way to do this is to set the personality >> to s390 (32 bit) and let the configure script do the right thing. >> Having a GCC which defaults to -m64 also requires to get a -m31 somehow >> into the option list. >> >> With that patch the GCC driver checks the current personality setting in >> order to figure out at runtime what the default is supposed to be. >> >> Bootstrapped and regtested on s390x. >> >> I will commit the patch after waiting a few days for comments. > > I don't like this, it is inconsistent with how it is handled on all other > targets and how it worked for years. Having setarch magically change the > default is dangerous.
Wow, that was quick ;) I didn't expect this to break things. Could you please elaborate where this might cause trouble? -Andreas- > >> 2018-03-12 Andreas Krebbel <kreb...@linux.vnet.ibm.com> >> >> * config/s390/driver-native.c (s390_host_detect_target_bits): New >> function. >> * config/s390/s390.h: Invoke s390_host_detect_target_bits if >> neither -m31 nor -m64 has been specified. > > Jakub >