[llvm-commits] [llvm] r125765 - in /llvm/trunk/lib/Target/X86: InstPrinter/X86InstComments.cpp Makefile Utils/ Utils/CMakeLists.txt Utils/Makefile Utils/X86ShuffleDecode.cpp Utils/X86ShuffleDecode.h X86ISelLowering.cpp X86ShuffleDecode.h

[Chandler Carruth]

On Thu, Feb 17, 2011 at 11:18 AM, David Greene <greened at obbligato.org>wrote:

> Author: greened
> Date: Thu Feb 17 13:18:59 2011
> New Revision: 125765
>
> URL: http://llvm.org/viewvc/llvm-project?rev=125765&view=rev
> Log:
>
> [AVX] Recorganize X86ShuffleDecode into its own library
> (LLVMX86Utils.a) to break cyclic library dependencies between
> LLVMX86CodeGen.a and LLVMX86AsmParser.a.  Previously this code was in
> a header file and marked static but AVX requires some additional
> functionality here that won't be used by all clients.  Since including
> unused static functions causes a gcc compiler warning, keeping it as a
> header would break builds that use -Werror.  Putting this in its own
> library solves both problems at once.
>
> Added:
>    llvm/trunk/lib/Target/X86/Utils/
>    llvm/trunk/lib/Target/X86/Utils/CMakeLists.txt
>

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. =/
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20110217/afc662bb/attachment.html>