[FFmpeg-devel] [PATCH] configure: detect 64-bit generic platforms

[FFmpeg-devel] [PATCH] configure: detect 64-bit generic platforms

[Rémi Denis-Courmont]

Currently, any unrecognised platform is treated as 32-bit. This should
detect *most* 64-bit platforms, namely LP64 and LLP64 ones.
Unfortunately this will not work for ILP32 ABIs on 64-bit ISAs, but
still better than nothing.
---
 configure | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/configure b/configure
index 83284427df..24eb051713 100755
--- a/configure
+++ b/configure
@@ -5599,6 +5599,12 @@ elif enabled x86; then
         ;;
     esac
 
+else
+
+    if test_cpp_condition inttypes.h "UINTPTR_MAX == UINT64_MAX"; then
+        enable fast_64bit
+    fi
+
 fi
 
 if [ "$cpu" != generic ]; then
-- 
2.45.1

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".

Re: [FFmpeg-devel] [PATCH] configure: detect 64-bit generic platforms

[Rémi Denis-Courmont]

Le tiistaina 11. kesäkuuta 2024, 19.48.19 EEST Rémi Denis-Courmont a écrit :
> Currently, any unrecognised platform is treated as 32-bit. This should
> detect *most* 64-bit platforms, namely LP64 and LLP64 ones.
> Unfortunately this will not work for ILP32 ABIs on 64-bit ISAs, but
> still better than nothing.
> ---
>  configure | 6 ++++++
>  1 file changed, 6 insertions(+)
> 
> diff --git a/configure b/configure
> index 83284427df..24eb051713 100755
> --- a/configure
> +++ b/configure
> @@ -5599,6 +5599,12 @@ elif enabled x86; then
>          ;;
>      esac
> 
> +else
> +
> +    if test_cpp_condition inttypes.h "UINTPTR_MAX == UINT64_MAX"; then

Should probably check >= rather than ==

> +        enable fast_64bit
> +    fi
> +
>  fi
> 
>  if [ "$cpu" != generic ]; then


-- 
Rémi Denis-Courmont
http://www.remlab.net/



_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".

Re: [FFmpeg-devel] [PATCH] configure: detect 64-bit generic platforms

[Sean McGovern]

Hi Remi,


On Tue, Jun 11, 2024, 13:06 Rémi Denis-Courmont <remi@remlab.net> wrote:

> Le tiistaina 11. kesäkuuta 2024, 19.48.19 EEST Rémi Denis-Courmont a écrit
> :
> > Currently, any unrecognised platform is treated as 32-bit. This should
> > detect *most* 64-bit platforms, namely LP64 and LLP64 ones.
> > Unfortunately this will not work for ILP32 ABIs on 64-bit ISAs, but
> > still better than nothing.
> > ---
> >  configure | 6 ++++++
> >  1 file changed, 6 insertions(+)
> >
> > diff --git a/configure b/configure
> > index 83284427df..24eb051713 100755
> > --- a/configure
> > +++ b/configure
> > @@ -5599,6 +5599,12 @@ elif enabled x86; then
> >          ;;
> >      esac
> >
> > +else
> > +
> > +    if test_cpp_condition inttypes.h "UINTPTR_MAX == UINT64_MAX"; then
>
> Should probably check >= rather than ==
>
> > +        enable fast_64bit
> > +    fi
> > +
> >  fi
> >
> >  if [ "$cpu" != generic ]; then
>
>
> --
> Rémi Denis-Courmont
> http://www.remlab.net/
>
>
>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel@ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
> To unsubscribe, visit link above, or email
> ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".
>

+1 for this. Let's get it in first before AVR32 & Alpha removal.

-- Sean McGovern

>
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".

Re: [FFmpeg-devel] [PATCH] configure: detect 64-bit generic platforms

[Rémi Denis-Courmont]

Le keskiviikkona 12. kesäkuuta 2024, 0.37.12 EEST Sean McGovern a écrit :
> +1 for this. Let's get it in first before AVR32 & Alpha removal.

AVR32 is not 64-bit; there are no deps one way or the other between the 
patches.

-- 
雷米‧德尼-库尔蒙
http://www.remlab.net/



_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".