<DIV>Hi,</DIV>
<DIV>Your patch is very nice to solve one of the problems, but not all.</DIV>
<DIV>There are two misses:</DIV>
<DIV>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.</DIV>
<DIV>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.</DIV>
<DIV>See my original attchment for more details.</DIV>
<DIV> </DIV>
<DIV>Thanks for your quick reply.</DIV>
<DIV>Rong Yan</DIV>
<DIV> </DIV>
<DIV>------------------
<DIV></DIV>
<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>
<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> Tue, Jan 12, 2016 02:47 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 when set--disable-asm</DIV></DIV>
<DIV><BR></DIV>On Mon, 11 Jan 2016 16:12:59 +0800, rongyan wrote:<BR>> Hi, Here attach a patch to fix the bug on POWER. When set<BR>> --disable-asm, the altivec can not be really disabled in original<BR>> version. With this patch, it works. Thanks. Rong Yan <BR>> <BR>Hi. As I understand it was only cosmetic bug of reporing:<BR>x264 [info]: using cpu capabilities: Altivec<BR>while none was really compiled or used.<BR><BR>As I understand my attached patch is enough to fix it.<BR>Report if it fix your problem (I can't really test PowerPC).<BR>
<DIV></DIV></DIV>