davide marked an inline comment as done. davide added inline comments.
================ Comment at: lldb/include/lldb/Host/Host.h:231 + /// Check whether a process is translated (Rosetta). + /// \arg process_info The info structure for the process queried. ---------------- aprantl wrote: > Is this supposed to be a generic function call that, e.g, should fire for > debugging a process in QEMU under Linux, or is it meant to specifically check > for Rosetta on macOS? The comment makes it sound like it's the latter and the > API name hints at the former. It would be nice to clarify this in the comment. It is meant specifically for Rosetta, but this leaves in `Host`, so I decided for a generic name. Do you have a better suggestion for the name or the comment ? Happy to go with it CHANGES SINCE LAST ACTION https://reviews.llvm.org/D82813/new/ https://reviews.llvm.org/D82813 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits