JDevlieghere marked an inline comment as done. JDevlieghere added inline comments.
================ Comment at: lldb/cmake/modules/FindLibXml28.cmake:14 + if (APPLE) + set(LIBXML2_LIBRARIES xml2) + endif() ---------------- kwk wrote: > labath wrote: > > Why is this under `if(APPLE)` ? > To me this looks as if this module file `FileLibXml28.cmake` is only relevant > for Apple. In line 10 it falls back to the standard CMake find-module file > (`find_package(LibXml2 QUIET)`). Is this correct? I'm not sure, I just kept the old behavior. I'm pretty sure it doesn't matter. Do you prefer to make it unconditional or just remove it altogether? Repository: rLLDB LLDB CHANGES SINCE LAST ACTION https://reviews.llvm.org/D72290/new/ https://reviews.llvm.org/D72290 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
