[PATCH] D82276: Make ninja smart console builds more pretty

Mehdi AMINI via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Jun 21 21:09:50 PDT 2020


mehdi_amini accepted this revision.
mehdi_amini added a comment.
This revision is now accepted and ready to land.

OK I see now, the description wasn't clear to me earlier. So it is about the fact that `find_package` failure is multiple-lines long, and it is called separately from LLVM, clang, lld, and lldb. Instead we will still display 4 times the warning but it'll be one line instead, right?

Seems like we could also call this once only from the LLVM CMakeLists.txt and make the variables available to the subprojects?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D82276/new/

https://reviews.llvm.org/D82276





More information about the llvm-commits mailing list