[Lldb-commits] [PATCH] D73938: [Host.mm] Check for the right macro instead of inlining it.

Jason Molenda via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Tue Feb 4 14:32:49 PST 2020


jasonmolenda added a comment.

I don't know about checking for NO_XPC_SERVICES for these -- they're LaunchServices/AppleScript related.  As Vedant mentioned with the earlier check, I think `TARGET_OS_EMBEDDED` rather than an architecture check would be the most correct change; these services are not available on iOS etc.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D73938





More information about the lldb-commits mailing list