* [PATCH] configure: Use -mlittle-endian instead of -mlittle for ppc64
@ 2022-01-19 9:56 mrezanin
2022-01-19 11:13 ` Paolo Bonzini
0 siblings, 1 reply; 3+ messages in thread
From: mrezanin @ 2022-01-19 9:56 UTC (permalink / raw)
To: qemu-devel; +Cc: pbonzini, richard.henderson
From: Miroslav Rezanina <mrezanin@redhat.com>
GCC options -mlittle and -mlittle-endian are equivalent on ppc64
architecture. However, clang supports only -mlittle-endian option.
Use longer form in configure to properly support both GCC and clang
compiler.
Signed-off-by: Miroslav Rezanina <mrezanin@redhat.com>
---
configure | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/configure b/configure
index e1a31fb332..e63c78ca67 100755
--- a/configure
+++ b/configure
@@ -658,7 +658,7 @@ case "$cpu" in
CPU_CFLAGS="-m64 -mbig" ;;
ppc64le)
cpu="ppc64"
- CPU_CFLAGS="-m64 -mlittle" ;;
+ CPU_CFLAGS="-m64 -mlittle-endian" ;;
s390)
CPU_CFLAGS="-m31" ;;
--
2.27.0
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] configure: Use -mlittle-endian instead of -mlittle for ppc64
2022-01-19 9:56 [PATCH] configure: Use -mlittle-endian instead of -mlittle for ppc64 mrezanin
@ 2022-01-19 11:13 ` Paolo Bonzini
2022-01-19 17:28 ` Philippe Mathieu-Daudé via
0 siblings, 1 reply; 3+ messages in thread
From: Paolo Bonzini @ 2022-01-19 11:13 UTC (permalink / raw)
To: mrezanin, qemu-devel; +Cc: richard.henderson
On 1/19/22 10:56, mrezanin@redhat.com wrote:
> From: Miroslav Rezanina <mrezanin@redhat.com>
>
> GCC options -mlittle and -mlittle-endian are equivalent on ppc64
> architecture. However, clang supports only -mlittle-endian option.
>
> Use longer form in configure to properly support both GCC and clang
> compiler.
>
> Signed-off-by: Miroslav Rezanina <mrezanin@redhat.com>
> ---
> configure | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/configure b/configure
> index e1a31fb332..e63c78ca67 100755
> --- a/configure
> +++ b/configure
> @@ -658,7 +658,7 @@ case "$cpu" in
> CPU_CFLAGS="-m64 -mbig" ;;
> ppc64le)
> cpu="ppc64"
> - CPU_CFLAGS="-m64 -mlittle" ;;
> + CPU_CFLAGS="-m64 -mlittle-endian" ;;
>
> s390)
> CPU_CFLAGS="-m31" ;;
Does -mbig need the same one line above?
Paolo
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] configure: Use -mlittle-endian instead of -mlittle for ppc64
2022-01-19 11:13 ` Paolo Bonzini
@ 2022-01-19 17:28 ` Philippe Mathieu-Daudé via
0 siblings, 0 replies; 3+ messages in thread
From: Philippe Mathieu-Daudé via @ 2022-01-19 17:28 UTC (permalink / raw)
To: Paolo Bonzini, mrezanin, qemu-devel; +Cc: richard.henderson
On 1/19/22 12:13, Paolo Bonzini wrote:
> On 1/19/22 10:56, mrezanin@redhat.com wrote:
>> From: Miroslav Rezanina <mrezanin@redhat.com>
>>
>> GCC options -mlittle and -mlittle-endian are equivalent on ppc64
>> architecture. However, clang supports only -mlittle-endian option.
>>
>> Use longer form in configure to properly support both GCC and clang
>> compiler.
>>
>> Signed-off-by: Miroslav Rezanina <mrezanin@redhat.com>
>> ---
>> configure | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/configure b/configure
>> index e1a31fb332..e63c78ca67 100755
>> --- a/configure
>> +++ b/configure
>> @@ -658,7 +658,7 @@ case "$cpu" in
>> CPU_CFLAGS="-m64 -mbig" ;;
>> ppc64le)
>> cpu="ppc64"
>> - CPU_CFLAGS="-m64 -mlittle" ;;
>> + CPU_CFLAGS="-m64 -mlittle-endian" ;;
>> s390)
>> CPU_CFLAGS="-m31" ;;
>
> Does -mbig need the same one line above?
Yes, and tests/tcg/configure.sh needs that change too.
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2022-01-19 17:32 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-01-19 9:56 [PATCH] configure: Use -mlittle-endian instead of -mlittle for ppc64 mrezanin
2022-01-19 11:13 ` Paolo Bonzini
2022-01-19 17:28 ` Philippe Mathieu-Daudé via
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.