On 2018-03-23 09:55, David Holmes wrote:
On 23/03/2018 6:46 PM, Magnus Ihse Bursie wrote:
On 2018-03-23 06:22, David Holmes wrote:
Hi Thomas,
On 23/03/2018 2:55 PM, Thomas Stüfe wrote:
Hi David,
would it not be pragmatic to accept Ao's patch - it looks fine to
me - since it certainly would not make matters worse. And let
Magnus follow up with a cleanup change later?
Well I hope Magnus's change is forthcoming.
It might be some time still. I'm working on a complete overhaul of
all CFLAGS and LDFLAGS, where this is a part of that picture, but I
was not planning on addressing just this thing urgently.
So, I think this patch will do for now. It solves the immediate
problem for MIPS, and I can come back and make a cleaner solution
later on.
Isn't the best quick fix one that only adds -m64 for x86? I recall a
report that arm32 is similarly broken.
Not really, because this is also needed on some other platforms, at
least s390x, as I recall. (This was the reason it was originally added.)
/Magnus
David
-----
AFAICS it's as easy to write this only for x86 as it is to exclude
it for non x86. Honestly I don't know why the Aarch64 patch was done
the way it was - there must be some subtlety here that I'm not aware
of.
I think it was just the smallest patch that worked for the aarch64
platform. I didn't spend time arguing about the fix, since it is
supposed to be short-lived anyway.
/Magnus