[Lldb-commits] [PATCH] D115211: [lldb] Make the LLDB version a first class citizen

[Nico Weber via Phabricator via lldb-commits]

thakis added a comment.

Thanks for this change! I always wondered why this strange lldbBase target existed in lldb/source instead of what's now done in this patch.



================
Comment at: lldb/source/CMakeLists.txt:4
-set(lldbBase_SOURCES
-    lldb.cpp
-  )
----------------
should lldb.cpp be deleted too?


================
Comment at: lldb/source/Version/Version.cpp:18
+#else
+  return "lldb version " CLANG_VERSION_STRING;
+#endif
----------------
does this intentionally use CLANG_VERSION_STRING instead of LLDB_VERSION_STRING?

(if not: remove clang/Basic/Version.h include too)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D115211