Page MenuHomePhabricator

Do not install lit-cpuid
ClosedPublic

Authored by sylvestre.ledru on Oct 5 2019, 12:21 AM.

Details

Summary

AFAIK, lit-cpuid is used by the tests.
Installing it causes LLVMExports*.cmake files to depend
on this program.
It causes some serious packaging issues as it would means
that llvm-dev depends on lldb.

See:
https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=941082
https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=941306

See also https://bugs.llvm.org/show_bug.cgi?id=43035
for a similar issue caused by
https://reviews.llvm.org/D56606

Event Timeline

mgorny accepted this revision.Oct 5 2019, 12:26 AM

WFM. You should really switch to stand-alone builds to avoid this kind of issues and help us support it better ;-P.

This revision is now accepted and ready to land.Oct 5 2019, 12:26 AM

Sure, happy to try, do you have some docs on this? Thanks :)

mgorny added a comment.Oct 5 2019, 1:32 AM

I'm afraid I don't. Learned all of it the hard way. But the minimal effort way is easy — you just install LLVM (with no other projects) first, then just cmake in specific other projects (clang, etc.). Getting tests to run usually requires having LLVM sources somewhere though.