[LLVMdev] Add ability for external projects use for buildinstalled version LLVM [PATCH]
Reid Spencer
reid at x10sys.com
Wed Feb 9 21:46:01 PST 2005
Vladimir,
I took a *very quick* look at your patch. I won't be able to look at it in more
detail until this weekend, but perhaps you can address a couple of concerns:
1. I don't understand the need for LLVMToolDirSlash. This seems like a
gratuitous change to me.
2. I don't think the change to CPP.Flags is right. It won't find the LLVM
header files
3. Removing the .PRECIOUS line will interfere with correct operation of
parallel (make -j N) builds.
4. I don't understand why LLVMUsedLibs and other variables should only be
set if LLVMLibDir is defined.
Before this change is committed, you need to test that all our scenarios, in
each combination:
1. Both with and without OBJ_DIR == SRC_DIR
2. Single and Parallel builds
3. Building LLVM versus building a project (and now building from installed)
Reid.
Vladimir Merzliakov wrote:
>>> Patch attached.
>>>
>>> This patch with header installation patch (
>>> http://mail.cs.uiuc.edu/pipermail/llvmdev/2005-February/003300.html )
>>> let external project (I am test it at my YAFL frontend for LLVM)
>>> build using installed LLVM version instead using LLVM obj/src dirs.
>>
>>
>> I think the patch looks basically sane. The one wierd thing I notice
>> is the replacement of mklib with libtool.
>
> Ops... I must be more careful. Updated patch attached.
>
> I use for build libtool "ltmain.sh (GNU libtool) 1.5.10" without problems.
> But I add code to preserve use $(LLVM_OBJ_ROOT)/mklib if LLVM_OBJ_ROOT set
> In other cases if $(LLVM_OBJ_ROOT)/mklib using important it installation
> can be added to LLVM/PROJ install process
>
>> However, this is definitely something Reid should review. I think
>> he's currently out of town but will be back next week.
>
> Waiting... :)
>
> Vladimir
>
>
> ------------------------------------------------------------------------
>
> _______________________________________________
> LLVM Developers mailing list
> LLVMdev at cs.uiuc.edu http://llvm.cs.uiuc.edu
> http://mail.cs.uiuc.edu/mailman/listinfo/llvmdev
More information about the llvm-dev
mailing list