Support of lldb on Kfreebsd

[sylvestre.ledru]

Summary

Patch by Pino Toscano. Reported in http://bugs.debian.org/835665

[brucem]

cmake/LLDBDependencies.cmake

168 ↗

(On Diff #69522)

This really should turn into a check so that we don't need this to be extended for every single OS that gets added.

[krytarowski]

cmake/LLDBDependencies.cmake

168 ↗

(On Diff #69522)

There is now support for CMake >= 3.0, which offers a builtin check for it.

[emaste]

• in the future can you please upload with context (e.g. git diff -U9999)
• I think the OS names in utilsOsType.py are kept sorted and the actual enum values are not part of an ABI

[tfiala]

cmake/LLDBDependencies.cmake

168 ↗

(On Diff #69522)

I agree with previous comments - if we can convert this to a CMake built-in check, that would be ideal.

[tfiala]

@sylvestre.ledru, I think this will be ready to go if you can convert that check to a CMake built-in check. Maybe you can pass that along to Pino Toscano?

[emaste]

Two requested changes called out in comments above

[sylvestre.ledru]

Updated by Pino!

[tfiala]

cmake/modules/LLDBConfig.cmake

414 ↗

(On Diff #72758)

Hi Sylvestre!

It's hard to tell without more context, but it looks like this location has most/all configurations going through it. For OSes that don't actually have a backtrace package, I think this emits a cmake error, doesn't it?

This might need to be protected by the systems that need the backtrace package. (Probably Unix-like systems only?)

[krytarowski]

cmake/modules/LLDBConfig.cmake

414 ↗

(On Diff #72758)

It's sufficient to drop the REQUIRED keyword and it will be fine. ${Backtrace_LIBRARY} will be evaluated to NIL in case of lack of this dependency.

I'm unsure whether find_pacakge() might be after usage of ${Backtrace_LIBRARY}. I would reorder it at least for clarity for a reader.

[tfiala]

Okay. I think we need a minor tweak, to drop the REQUIRED on the Backtrace usage.

[tfiala]

Looks reasonable to me.

@beanz, any comments on this?

[beanz]

Looks fine to me.

lldb/trunk/cmake/LLDBDependencies.cmake

if(LLVM_ENABLE_TERMINFO AND HAVE_TERMINFO)
     list(APPEND LLDB_SYSTEM_LIBS ${TERMINFO_LIBS})
   endif()
 endif()
 
 if (NOT HAVE_CXX_ATOMICS64_WITHOUT_LIB )
     list(APPEND LLDB_SYSTEM_LIBS atomic)
 endif()
 
-# On FreeBSD/NetBSD backtrace() is provided by libexecinfo, not libc.
-if (CMAKE_SYSTEM_NAME MATCHES "FreeBSD" OR CMAKE_SYSTEM_NAME MATCHES "NetBSD")
-  list(APPEND LLDB_SYSTEM_LIBS execinfo)
-endif()
+list(APPEND LLDB_SYSTEM_LIBS ${Backtrace_LIBRARY})
 
 if (NOT LLDB_DISABLE_PYTHON AND NOT LLVM_BUILD_STATIC)
   list(APPEND LLDB_SYSTEM_LIBS ${PYTHON_LIBRARIES})
 endif()
 
 list(APPEND LLDB_SYSTEM_LIBS ${system_libs})
 
 if (LLVM_BUILD_STATIC)

lldb/trunk/cmake/modules/LLDBConfig.cmake

 if(MSVC)
     set(LLDB_USE_BUILTIN_DEMANGLER ON)
 else()
     option(LLDB_USE_BUILTIN_DEMANGLER "Use lldb's builtin demangler instead of the system one" ON)
 endif()
 if(LLDB_USE_BUILTIN_DEMANGLER)
     add_definitions(-DLLDB_USE_BUILTIN_DEMANGLER)
 endif()
+
+find_package(Backtrace)

lldb/trunk/scripts/Python/modules/CMakeLists.txt

 # Disable some warnings triggered by Python's headers.
 check_cxx_compiler_flag("-Wno-macro-redefined"
                         CXX_SUPPORTS_NO_MACRO_REDEFINED)
 if (CXX_SUPPORTS_NO_MACRO_REDEFINED)
   set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wno-macro-redefined")
 endif ()
 
 # build the Python readline suppression module only on Linux
-if (CMAKE_SYSTEM_NAME MATCHES "Linux")
+!if (CMAKE_SYSTEM_NAME MATCHES "Linux" OR CMAKE_SYSTEM_NAME STREQUAL "GNU" OR CMAKE_SYSTEM_NAME STREQUAL "kFreeBSD")
    add_subdirectory(readline)
 endif()

lldb/trunk/scripts/utilsOsType.py

if sys.version_info.major >= 3:
 
     class EnumOsType(Enum):
         Unknown = 0
         Darwin = 1
         FreeBSD = 2
         Linux = 3
         NetBSD = 4
         Windows = 5
+        kFreeBSD = 6
 else:
     class EnumOsType(object):
         values = ["Unknown",
                   "Darwin",
                   "FreeBSD",
                   "Linux",
                   "NetBSD",
-                  "Windows"]
+                  "Windows",
+                  "kFreeBSD"]
 
         class __metaclass__(type):
             #++----------------------------------------------------------------
             # Details:  Fn acts as an enumeration.
             # Args:     vName - (R) Enumeration to match.
             # Returns:  Int - Matching enumeration/index.
             # Throws:   None.
             #--
def determine_os_type():
     elif strOS.startswith("freebsd"):
         eOSType = EnumOsType.FreeBSD
     elif strOS.startswith("linux"):
         eOSType = EnumOsType.Linux
     elif strOS.startswith("netbsd"):
         eOSType = EnumOsType.NetBSD
     elif strOS == "win32":
         eOSType = EnumOsType.Windows
+    elif strOS.startswith("gnukfreebsd"):
+        eOSType = EnumOsType.kFreeBSD
 
     return eOSType