Re: [FFmpeg-devel] [PATCH] arm: only enable setend on ARMv6

2015-06-05 Thread Andreas Cadhalpun
On 05.06.2015 16:29, Michael Niedermayer wrote: > On Fri, Jun 05, 2015 at 01:40:31PM +0200, Andreas Cadhalpun wrote: >> On 05.06.2015 00:47, Michael Niedermayer wrote: >>> or (AV_CPU_FLAG_ARMV6T2 | AV_CPU_FLAG_ARMV6) >> >> That seems better, assuming setend is useful on ARMv6t2. > > AFAIK its even

Re: [FFmpeg-devel] [PATCH] arm: only enable setend on ARMv6

2015-06-05 Thread Michael Niedermayer
On Fri, Jun 05, 2015 at 01:40:31PM +0200, Andreas Cadhalpun wrote: > On 05.06.2015 00:47, Michael Niedermayer wrote: > > On Thu, Jun 04, 2015 at 11:17:45PM +0200, Andreas Cadhalpun wrote: > >> Without this check it causes SIGILL crashes on ARMv5. > >> > >> Signed-off-by: Andreas Cadhalpun > >> ---

Re: [FFmpeg-devel] [PATCH] arm: only enable setend on ARMv6

2015-06-05 Thread Andreas Cadhalpun
On 05.06.2015 00:47, Michael Niedermayer wrote: > On Thu, Jun 04, 2015 at 11:17:45PM +0200, Andreas Cadhalpun wrote: >> Without this check it causes SIGILL crashes on ARMv5. >> >> Signed-off-by: Andreas Cadhalpun >> --- >> libavutil/arm/cpu.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(

Re: [FFmpeg-devel] [PATCH] arm: only enable setend on ARMv6

2015-06-04 Thread Michael Niedermayer
On Thu, Jun 04, 2015 at 11:17:45PM +0200, Andreas Cadhalpun wrote: > Without this check it causes SIGILL crashes on ARMv5. > > Signed-off-by: Andreas Cadhalpun > --- > libavutil/arm/cpu.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavutil/arm/cpu.c b/libavutil/ar

[FFmpeg-devel] [PATCH] arm: only enable setend on ARMv6

2015-06-04 Thread Andreas Cadhalpun
Without this check it causes SIGILL crashes on ARMv5. Signed-off-by: Andreas Cadhalpun --- libavutil/arm/cpu.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavutil/arm/cpu.c b/libavutil/arm/cpu.c index f1683e8..ac42a05 100644 --- a/libavutil/arm/cpu.c +++ b/libavutil/arm