<DIV>Hi,</DIV>
<DIV>I am sorry, yesterday I just looked at the patch but not compiled. I tried it just now, it perfectly solved the problem.</DIV>
<DIV>Thanks.</DIV>
<DIV> </DIV>
<DIV>Rong Yan</DIV>
<DIV>
<DIV style="FONT-SIZE: 12px; FONT-FAMILY: Arial Narrow; COLOR: #909090">------------------</DIV>
<DIV style="FONT-SIZE: 14px; FONT-FAMILY: Verdana; COLOR: #000">
<DIV>The world has enough for everyone's need, but not enough for everyone's greed.</DIV></DIV></DIV>
<DIV> </DIV>
<DIV>
<DIV><BR></DIV>
<DIV><BR></DIV>
<DIV style="FONT-SIZE: 12px; FONT-FAMILY: Arial Narrow; PADDING-BOTTOM: 2px; PADDING-TOP: 2px; PADDING-LEFT: 0px; PADDING-RIGHT: 0px">------------------ Original ------------------</DIV>
<DIV style="FONT-SIZE: 12px; BACKGROUND: #efefef; PADDING-BOTTOM: 8px; PADDING-TOP: 8px; PADDING-LEFT: 8px; PADDING-RIGHT: 8px">
<DIV><B>From: </B> "BugMaster";<BugMaster@narod.ru>;</DIV>
<DIV><B>Date: </B> Wed, Jan 13, 2016 01:22 AM</DIV>
<DIV><B>To: </B> "Mailing list for x264 developers"<x264-devel@videolan.org>; <WBR></DIV>
<DIV></DIV>
<DIV><B>Subject: </B> Re: [x264-devel] patch[1/1]: fix bug to disable altivec whenset--disable-asm</DIV></DIV>
<DIV><BR></DIV>On Tue, 12 Jan 2016 15:46:44 +0800, rongyan wrote:<BR>>  Hi,<BR>>  Your patch is very nice to solve one of the problems, but not all.<BR><BR>Have you even tried it (compiled/run it)?<BR><BR>>  There are two misses:<BR>>  1. in configure file, need to modify "=" to "==" in line 715. It is to fix the bug when --disable-asm is added, the altivec optimizaiton can be really disabled.<BR><BR>"=" and "==" are equal in bash (this is not C).<BR>And according to http://www.gnu.org/software/bash/manual/html_node/Bash-Conditional-Expressions.html#Bash-Conditional-Expressions "=" form is more POSIX conformant.<BR><BR>>  2. in common/cpu.c, beside your modification, also need to look at the branch: #elif SYS_LINUX, when HAVE_ALTIVEC is disable, need to return 0. It is to fix the reporting problem.<BR>>  See my original attchment for more details.<BR><BR>"#elif SYS_LINUX" - is continuation of "#if SYS_MACOSX || SYS_OPENBSD || SYS_FREEBSD" which is already inside "#elif ARCH_PPC" which I changed.<BR><BR>_______________________________________________<BR>x264-devel mailing list<BR>x264-devel@videolan.org<BR>https://mailman.videolan.org/listinfo/x264-devel<BR>
<DIV></DIV></DIV>