Re: [ITA] llvm

On Sat, 10 May 2025, Takashi Yano via Cygwin-apps wrote:

> On Fri, 9 May 2025 13:48:00 -0700 (PDT)
> Jeremy Drake wrote:
> > On Sat, 10 May 2025, Takashi Yano via Cygwin-apps wrote:
> >
> > > Sorry, I don't understand why it is necessary even for
> > > bootstrapping. Could you please let me know the reason
> > > why you think the test should be enabled for bootstrapping?
> >
> > I got an error because usr/lib/libllvm*.a wasn't present if it didn't
> > build and install libllvm_gtest.a, and it didn't if
> > LLVM_INCLUDE_TESTS=OFF.  I guess the libllvm_devel_CONTENTS could exclude
> > usr/lib/libllvm*.a if BOOTSTRAP=yes, I just chose to include instead.
>
> As far as I tested, usr/lib/libllvm*.a is created if LLVM_INSTALL_GTEST=ON
> is set even with LLVM_INCLUDE_TESTS=OFF.

If that works, great.  It didn't work for me when I was trying, but that
may have been an artifact of not starting the build from scratch when
testing.  I don't expect it makes much difference either way though,
LLVM_INCLUDE_TESTS is documented as only including the *targets* for the
tests, and LLVM_BUILD_TESTS controlls whether they are built by default
(or only via the check targets when tests are requested to be run).