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 <[email protected]> > --- > 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/cpu.c > @@ -128,7 +128,7 @@ int ff_get_cpu_flags_arm(void) > trickle down. */ > if (flags & (AV_CPU_FLAG_VFPV3 | AV_CPU_FLAG_NEON)) > flags |= AV_CPU_FLAG_ARMV6T2; > - else > + else if (flags & AV_CPU_FLAG_ARMV6) > /* Some functions use the 'setend' instruction which is deprecated on > ARMv8 > * and serializing on some ARMv7 cores. This ensures such functions > * are only enabled on ARMv6. */
shouldnt this be after
"- if (flags & AV_CPU_FLAG_ARMV6T2)
flags |= AV_CPU_FLAG_ARMV6;
"
or (AV_CPU_FLAG_ARMV6T2 | AV_CPU_FLAG_ARMV6)
[...]
--
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
If a bugfix only changes things apparently unrelated to the bug with no
further explanation, that is a good sign that the bugfix is wrong.
signature.asc
Description: Digital signature
_______________________________________________ ffmpeg-devel mailing list [email protected] http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
