llvm-zorg icon indicating copy to clipboard operation
llvm-zorg copied to clipboard

Add new builder configuration to test use of LLVM dylib across all tools

Open serge-sans-paille opened this issue 1 year ago • 5 comments

serge-sans-paille avatar Jul 12 '23 09:07 serge-sans-paille

This is an alternate approach to #37, I think there's value in testing both config.

serge-sans-paille avatar Jul 12 '23 09:07 serge-sans-paille

What's being tested with the compiler-rt build?

tstellar avatar Jul 12 '23 13:07 tstellar

On Wed, Jul 12, 2023 at 01:27:28PM +0000, Tom Stellard wrote:

What's being tested with the compiler-rt build?

I can't tell exactly, but compiler-rt/test contains a lot of reference to %llvm* and %clang* that are going to use libLLVM.

serge-sans-paille avatar Jul 12 '23 14:07 serge-sans-paille

@serge-sans-paille What's the advantage of having two separate builders?

tstellar avatar Jul 13 '23 09:07 tstellar

The patch itself looks fine. But the question https://github.com/llvm/llvm-zorg/pull/39#issuecomment-1633855171 needs the answer.

gkistanova avatar Sep 13 '23 18:09 gkistanova