On Thu, May 24, 2012 at 01:45:41PM -0400, Michael Meissner wrote: > This alternative patch just disables building the 32-bit softfloat multlib, > and > removes the -mstrict-align in the powerpc64-linux case. > > I have bootstrapped it and had no regressions. > > David, which patch do you prefer? > > 2012-05-24 Michael Meissner <meiss...@linux.vnet.ibm.com> > > * config/rs6000/t-linux64: Delete the 32-bit multilib that uses > software floating point emulation. No longer build the multilibs > with -mstrict-align.
I like this patch. I like it because all uses of MULTILIB_EXTRA_OPTS are suspect. Why? The option is not applied to the default multilib! That means that prior to this patch you get different libraries depending on the compiler default. For example, a powerpc-linux gcc defaulting to -m32 builds the 32-bit libraries without -mstrict-align, while a powerpc64-linux gcc defaulting to -m64 builds the 32-bit libraries with -mstrict-align. This is confusing to say the least, and I recall seeing libstdc++ test failures due to the difference. I've had "MULTILIB_EXTRA_OPTS =" in my t-linux64 for quite some time. -- Alan Modra Australia Development Lab, IBM
