<div class="gmail_quote">On Thu, Feb 17, 2011 at 11:18 AM, David Greene <span dir="ltr"><<a href="mailto:greened@obbligato.org">greened@obbligato.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<div id=":su">Author: greened<br>
Date: Thu Feb 17 13:18:59 2011<br>
New Revision: 125765<br>
<br>
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=125765&view=rev" target="_blank" class="cremed">http://llvm.org/viewvc/llvm-project?rev=125765&view=rev</a><br>
Log:<br>
<br>
[AVX] Recorganize X86ShuffleDecode into its own library<br>
(LLVMX86Utils.a) to break cyclic library dependencies between<br>
LLVMX86CodeGen.a and LLVMX86AsmParser.a.  Previously this code was in<br>
a header file and marked static but AVX requires some additional<br>
functionality here that won't be used by all clients.  Since including<br>
unused static functions causes a gcc compiler warning, keeping it as a<br>
header would break builds that use -Werror.  Putting this in its own<br>
library solves both problems at once.<br>
<br>
Added:<br>
    llvm/trunk/lib/Target/X86/Utils/<br>
    llvm/trunk/lib/Target/X86/Utils/CMakeLists.txt</div></blockquote></div><br><div>While you added the CMake target for this, you didn't update the dependencies of the X86 targets that use this... I now get link failures on my builds. =/</div>