[Lldb-commits] [PATCH] D82813: [Apple Silicon] Rewrite part of the Rosetta support to be confined in Apple specific files
Jonas Devlieghere via Phabricator via lldb-commits
lldb-commits at lists.llvm.org
Mon Jun 29 16:54:52 PDT 2020
JDevlieghere added inline comments.
================
Comment at: lldb/source/Host/macosx/objcxx/Host.mm:1471
+ size_t bufsize = sizeof(processInfo);
+ if (sysctl(mib, (unsigned)(sizeof(mib)/sizeof(int)), &processInfo,
+ &bufsize, NULL, 0) == 0 && bufsize <= 0)
----------------
nit: `static_cast` for consistency with line 1468?
================
Comment at: lldb/source/Host/macosx/objcxx/Host.mm:1472
+ if (sysctl(mib, (unsigned)(sizeof(mib)/sizeof(int)), &processInfo,
+ &bufsize, NULL, 0) == 0 && bufsize <= 0)
+ return false;
----------------
nit: `nullptr`
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D82813/new/
https://reviews.llvm.org/D82813
More information about the lldb-commits
mailing list