[Lldb-commits] [PATCH] D60180: [CMake] Don't explicitly use LLVM_LIBRARY_DIR in standalone builds

Stefan Gränitz via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Fri Apr 5 08:02:02 PDT 2019

sgraenitz added a comment.

> [LLVM_LIBRARY_DIR is] not a cache variable when it's an in-tree build

Great, then please go ahead.

> From what I remember, you added back [...]

Yes this was a quick-fix. I didn't have the patience in that moment to check each of them individually.

> If there's a situation where any of these variables don't get propagated from the LLVMConfig, I'd like to know about it so I can add comments above these variables explaining why these variables are set since it's non-obvious just from grepping the lldb tree.

It's usually a good idea to check both, the LLVMConfig.cmake from a build-tree and from an install-tree. They are quite different and LLDB standalone should work with both.

You can check the green dragon bot that we added recently: http://green.lab.llvm.org/green/view/LLDB/job/lldb-cmake-standalone/
It builds against both versions (and BTW it also dumps CMake cache entries).



More information about the lldb-commits mailing list