[PATCH] D102732: [CMake] Don't LTO optimize targets that aren't part of any distribution

[Justin Bogner via Phabricator via llvm-commits]

bogner added inline comments.


================
Comment at: llvm/CMakeLists.txt:934
 
+include(LLVMDistributionSupport)
+
----------------
phosek wrote:
> smeenai wrote:
> > Is this moved up just to get the distribution target map built in time for the new use?
> Yes, I need `LLVM_DISTRIBUTION_FOR_*` to be populated before creating any targets.
I'm seeing test failures if I do `ninja distribution test-depends` and then `ninja check` and I suspect moving this is related.  It's an llvm-config test that fails because `lib/libLLVMObjCMetadata.a` wasn't built though, so it's entirely possible that moving this just exposed a missing dependency somewhere else.

Do you happen to have any insight here?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D102732