Problem seen with Add LLVMBuild for ObjectYAML

[Dan Palermo via llvm-commits]

Rafael,

I think I see what is going wrong.  Internally we picked up this patch, but we have not yet merged a previous patch from you (http://reviews.llvm.org/D17771) to Move ObjectYAML code to a new library.  We sometimes get patches merged out of order internally when there are patch conflicts with local changes that need to be resolved.

You can ignore my previous message.

Sorry for the interruption,
Dan

--

Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, 
hosted by The Linux Foundation

-----Original Message-----
From: Dan Palermo [mailto:dpalermo at codeaurora.org] 
Sent: Wednesday, March 02, 2016 12:10 PM
To: 'Rafael Espíndola'
Cc: 'Chris Bieneman'; 'llvm-commits'
Subject: RE: Problem seen with Add LLVMBuild for ObjectYAML

With the original patch back in place, and after removing the build dir & rerunning cmake, I still see the same behavior....no libLLVMObjectYAML.a is present anywhere under llvm.  I also don't see a build/lib/ObjectYAML directory while I do have a build/lib/Object directory (in case that is another clue).

Could you have other uncommitted files in your tree by chance that are required to get this library to build?

Also, attached is my cmake command in case there is something interacting badly with this change.

Dan

-----Original Message-----
From: Rafael Espíndola [mailto:rafael.espindola at gmail.com]
Sent: Wednesday, March 02, 2016 11:55 AM
To: Dan Palermo
Cc: Chris Bieneman; llvm-commits
Subject: Re: Problem seen with Add LLVMBuild for ObjectYAML

That is so odd. I do get a lib/libLLVMObjectYAML.a. Can you try rm -rf the build dir and running cmake agin?

Cheers,
Rafael


On 2 March 2016 at 12:47, Dan Palermo <dpalermo at codeaurora.org> wrote:
> Rafael,
>
>
>
> I am working with the shared library build of LLVM to build Halide and 
> our build broke after this change came in:
>
>
>
> Category        None
> Changed by      Rafael Espindola <rafael.espindola at gmail.com>
> Changed at      Wed 02 Mar 2016 06:18:31
> Repository
> ssh://review-hexagon.quicinc.com:29418/llvm/llvm-mainline.git
> Project         llvm
> Branch  master
> Revision        a2ff97d4218d0ca8eac0ea6980d6373025084f27
> Comments
>
> Add LLVMBuild for ObjectYAML.
> Should fix the DBUILD_SHARED_LIBS bots.
>
> With this change, the Halide build fails with:
>
> ...
>
> ar: creating lib/libHalide.a
>
> ranlib lib/libHalide.a
>
> llvm-config: error: component libraries and shared library
>
>
>
> llvm-config: error: missing:
> /data/buildbot/slave/halide-80-build/src/llvm/build/lib/libLLVMObjectY
> AML.a
>
>
>
> The libLLVMObjectYAML.a library is not present in the llvm build/lib 
> or install/lib after a build of LLVM.  To make progress locally, I 
> just needed to remove ObjectYAML from lib/LLVMBuild.txt but I thought 
> I would let you know what I was seeing since it will probably affect others as well.
>
>
>
> Thanks,
>
> Dan Palermo
>
>
>
> --
>
> Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, 
> hosted by The Linux Foundation
>
>