Bug 1142586 - LTO: gstreamer-plugins-base build fails on aarch64
LTO: gstreamer-plugins-base build fails on aarch64
Status: IN_PROGRESS
Classification: openSUSE
Product: openSUSE Tumbleweed
Classification: openSUSE
Component: Basesystem
Current
Other Other
: P5 - None : Normal (vote)
: ---
Assigned To: Martin Liška
E-mail List
:
Depends on:
Blocks: 1133084
  Show dependency treegraph
 
Reported: 2019-07-24 07:57 UTC by Martin Liška
Modified: 2019-07-24 08:13 UTC (History)
1 user (show)

See Also:
Found By: ---
Services Priority:
Business Priority:
Blocker: ---
Marketing QA Status: ---
IT Deployment: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Comment 1 Martin Liška 2019-07-24 07:58:06 UTC
Fails due to a wrong configure detection:

  checking for sys/types.h... (cached) yes
  checking for sys/wait.h... yes
  checking for sys/stat.h... (cached) yes
- checking for ARM NEON support in current arch/CFLAGS... no
+ checking for ARM NEON support in current arch/CFLAGS... yes
  checking for stdint types... stdint.h (shortcircuit)
  make use of stdint.h in _stdint.h (assuming C99 compatible system)
  checking for localtime_r... yes
Comment 2 Guillaume GARDET 2019-07-24 08:00:57 UTC
(In reply to Martin Liška from comment #1)
> Fails due to a wrong configure detection:
> 
>   checking for sys/types.h... (cached) yes
>   checking for sys/wait.h... yes
>   checking for sys/stat.h... (cached) yes
> - checking for ARM NEON support in current arch/CFLAGS... no
> + checking for ARM NEON support in current arch/CFLAGS... yes
>   checking for stdint types... stdint.h (shortcircuit)
>   make use of stdint.h in _stdint.h (assuming C99 compatible system)
>   checking for localtime_r... yes

aarch64 does support NEON. So, it should be ok to enable NEON on aarch64.
Comment 3 Martin Liška 2019-07-24 08:07:40 UTC
(In reply to Guillaume GARDET from comment #2)
> (In reply to Martin Liška from comment #1)
> > Fails due to a wrong configure detection:
> > 
> >   checking for sys/types.h... (cached) yes
> >   checking for sys/wait.h... yes
> >   checking for sys/stat.h... (cached) yes
> > - checking for ARM NEON support in current arch/CFLAGS... no
> > + checking for ARM NEON support in current arch/CFLAGS... yes
> >   checking for stdint types... stdint.h (shortcircuit)
> >   make use of stdint.h in _stdint.h (assuming C99 compatible system)
> >   checking for localtime_r... yes
> 
> aarch64 does support NEON. So, it should be ok to enable NEON on aarch64.

I've got a patch candidate..
Comment 4 Martin Liška 2019-07-24 08:13:55 UTC
I've just created an upstream merge request:
https://gitlab.freedesktop.org/gstreamer/gst-plugins-base/merge_requests/338