Re: [PATCH] Recognize aarch64_ilp32

2016-12-06 Thread Earnie
On 12/6/2016 4:34 AM, Andreas Schwab wrote: > On Dez 06 2016, Ben Elliston wrote: > >> On Tue, Dec 06, 2016 at 09:17:50AM +0100, Andreas Schwab wrote: >> > So what's the replacement? Using system tools to discover the ABI. >>> >>> ??? That's what I did. >> >> Sorry, I meant "standar

Re: [PATCH] Recognize aarch64_ilp32

2016-12-06 Thread Andreas Schwab
On Dez 06 2016, Ben Elliston wrote: > On Tue, Dec 06, 2016 at 09:17:50AM +0100, Andreas Schwab wrote: > >> >> So what's the replacement? >> > >> > Using system tools to discover the ABI. >> >> ??? That's what I did. > > Sorry, I meant "standard system tools" that can reasonably be expected > to

Re: [PATCH] Recognize aarch64_ilp32

2016-12-06 Thread Ben Elliston
On Tue, Dec 06, 2016 at 09:17:50AM +0100, Andreas Schwab wrote: > >> So what's the replacement? > > > > Using system tools to discover the ABI. > > ??? That's what I did. Sorry, I meant "standard system tools" that can reasonably be expected to be installed. Many systems do not have a C compile

Re: [PATCH] Recognize aarch64_ilp32

2016-12-06 Thread Andreas Schwab
On Dez 05 2016, Ben Elliston wrote: > On Mon, Dec 05, 2016 at 02:07:58PM +0100, Andreas Schwab wrote: > >> > Please find another way to get the ilp32 status. I'm not accepting >> > new uses of set_cc_for_build, because it is an abomination. >> >> So what's the replacement? > > Using system tool

Re: [PATCH] Recognize aarch64_ilp32

2016-12-05 Thread Ben Elliston
On Mon, Dec 05, 2016 at 02:07:58PM +0100, Andreas Schwab wrote: > > Please find another way to get the ilp32 status. I'm not accepting > > new uses of set_cc_for_build, because it is an abomination. > > So what's the replacement? Using system tools to discover the ABI. It's fine to use portable

Re: [PATCH] Recognize aarch64_ilp32

2016-12-05 Thread Andreas Schwab
On Dez 05 2016, Jan Engelhardt wrote: > On Monday 2016-12-05 14:03, Ben Elliston wrote: >>On Mon, Dec 05, 2016 at 01:37:21PM +0100, Andreas Schwab wrote: >> >>> +aarch64:Linux:*:* | aarch64_be:Linux:*:*) >>> + eval $set_cc_for_build >>> + if echo __ILP32__ | $CC_FOR_BUILD -E - 2>/dev/null

Re: [PATCH] Recognize aarch64_ilp32

2016-12-05 Thread Andreas Schwab
On Dez 05 2016, Ben Elliston wrote: > Please find another way to get the ilp32 status. I'm not accepting > new uses of set_cc_for_build, because it is an abomination. So what's the replacement? Andreas. -- Andreas Schwab, SUSE Labs, sch...@suse.de GPG Key fingerprint = 0196 BAD8 1CE9 1970 F4

Re: [PATCH] Recognize aarch64_ilp32

2016-12-05 Thread Jan Engelhardt
On Monday 2016-12-05 15:28, Andreas Schwab wrote: >On Dez 05 2016, Jan Engelhardt wrote: > >> On Monday 2016-12-05 14:03, Ben Elliston wrote: >>>On Mon, Dec 05, 2016 at 01:37:21PM +0100, Andreas Schwab wrote: >>> +aarch64:Linux:*:* | aarch64_be:Linux:*:*) + eval $set_cc_for_build >

Re: [PATCH] Recognize aarch64_ilp32

2016-12-05 Thread Jan Engelhardt
On Monday 2016-12-05 14:03, Ben Elliston wrote: >On Mon, Dec 05, 2016 at 01:37:21PM +0100, Andreas Schwab wrote: > >> +aarch64:Linux:*:* | aarch64_be:Linux:*:*) >> +eval $set_cc_for_build >> +if echo __ILP32__ | $CC_FOR_BUILD -E - 2>/dev/null \ >> +| grep -q __ILP32__ >> +t

Re: [PATCH] Recognize aarch64_ilp32

2016-12-05 Thread Ben Elliston
On Mon, Dec 05, 2016 at 01:37:21PM +0100, Andreas Schwab wrote: > +aarch64:Linux:*:* | aarch64_be:Linux:*:*) > + eval $set_cc_for_build > + if echo __ILP32__ | $CC_FOR_BUILD -E - 2>/dev/null \ > + | grep -q __ILP32__ > + then > + : > + else > + UNAME_MAC

[PATCH] Recognize aarch64_ilp32

2016-12-05 Thread Andreas Schwab
* config.sub (aarch64_ilp32, aarch64_be_ilp32, aarch64_ilp32-*) (aarch64_be_ilp32-*): New. * config.guess (aarch64:Linux:*:*, aarch64_be:Linux:*:*): Unify. Append _ilp32 to UNAME_MACHINE if ILP32 ABI is detected. --- config.guess | 16 ++-- config.sub