[Lldb-commits] [PATCH] D75696: Correctly detect iOS simulator processes.

2020-03-06 Thread Adrian Prantl via Phabricator via lldb-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rG59d816d88464: Correctly identify iOS simulator processes in debugserver. (authored by aprantl). Herald added a project: LLDB. Changed prior to commit: https://reviews.llvm.org/D75696?vs=248634&id=248776

[Lldb-commits] [PATCH] D75696: Correctly detect iOS simulator processes.

2020-03-06 Thread Adrian Prantl via Phabricator via lldb-commits
aprantl marked an inline comment as done. aprantl added inline comments. Comment at: lldb/test/API/functionalities/gdb_remote_client/TestWasm.py:35 -if packet == "qProcessInfo": -return self.qProcessInfo() if packet[0:13] == "qRegisterInfo": -

[Lldb-commits] [PATCH] D75696: Correctly detect iOS simulator processes.

2020-03-05 Thread Jason Molenda via Phabricator via lldb-commits
jasonmolenda accepted this revision. jasonmolenda added a comment. This revision is now accepted and ready to land. Overall looks fine, Fred's comment is correct, nb you want to test it with `#if TARGET_OS_OSX == 1`, it's always defined either to 0 or 1. Comment at: lldb/test/

[Lldb-commits] [PATCH] D75696: Correctly detect iOS simulator processes.

2020-03-05 Thread Frederic Riss via Phabricator via lldb-commits
friss added inline comments. Comment at: lldb/tools/debugserver/source/MacOSX/MachProcess.mm:601-607 +static bool IsMacOSHost() { + static uint32_t g_host_cputype = 0; + size_t len = sizeof(uint32_t); + if (!g_host_cputype) +::sysctlbyname("hw.cputype", &g_host_cputype, &l

[Lldb-commits] [PATCH] D75696: Correctly detect iOS simulator processes.

2020-03-05 Thread Adrian Prantl via Phabricator via lldb-commits
aprantl updated this revision to Diff 248634. aprantl added a comment. Here is a more principled variant of the patch. I pushed the logic down to debugserver, so we don't need to guess or fixup anything in LLDB. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D75696/new/ https://reviews.l

[Lldb-commits] [PATCH] D75696: Correctly detect iOS simulator processes.

2020-03-05 Thread Adrian Prantl via Phabricator via lldb-commits
aprantl created this revision. aprantl added reviewers: jasonmolenda, friss. Herald added subscribers: aheejin, sbc100. This patch detects when an ios process runs on a macOS host and updates the target triple with the "simulator" environment. This is part of https://bugs.swift.org/browse/SR-1197