If you don't like the LTO changes, I can also drop them. The main purpose of this change is obviously to make the package build again. But ThinLTO has a very small overhead, so I thought why not? We are also using it in the upcoming llvm9.

Christophe Giboudeaux (cgiboudeaux) - about 5 years ago

reviewer

Yes please, I'd rather avoid this change.

About the patch: Clazy 1.6 will be out in a couple days and the build fix for clang 9 is different (it's linking against clangToolingRefactoring instead of clangToolingRefactor). See https://cgit.kde.org/clazy.git/commit/?id=902da94

---

Aaron Puchert (aaronpuchert) - about 5 years ago

author target maintainer

OK, then I'll remove the ThinLTO changes again.

Just to be clear: the upcoming libclang9 package does not contain any Clang component libraries, just libclang.so.9 and libclang-cpp.so.9. So the patch is still needed. This is because of https://releases.llvm.org/9.0.0/tools/clang/docs/ReleaseNotes.html#build-system-changes.