[LLVMdev] [lld] Patch to incorporate lld test-suite into "make check-all"

Michael Spencer bigcheesegs at gmail.com
Tue Jul 31 11:46:59 PDT 2012


On Tue, Jul 31, 2012 at 2:35 AM, Andy Gibbs <andyg1001 at hotmail.co.uk> wrote:
> Hi again,
>
> Any interest in this patch to incorporate lld testing into "make check-all"
> when lld built as part of llvm?  Is this the best place to post patches for
> lld?
>
> I've attached it again here to include a missing dependency.
>
> Cheers
> Andy
>
>
> On Thursday, July 26, 2012 1:45 PM, Andy Gibbs wrote:
>> Hi,
>>
>> Attached is a patch which incorporates the lld test-suite into llvm's "make
>> check-all" if lld is built within llvm.  Please could someone review this
>> for me.
>>
>> I've hand-amended the patch so that it should be easier to review.  The
>> majority of the patch is simply changing the indent once inside and else()/
>> endif() block.  Beyond that, I've standardised the test target name to
>> "check-lld", but retained the legacy "lld-test" target.
>>
>> Regards
>> Andy

Committed as r161067. Thanks!

Can you create futures patches relative to the lld directory instead
of llvm? I don't have lld checked out inside llvm.

- Michael Spencer



More information about the llvm-dev mailing list