[Lldb-commits] [PATCH] D124573: [trace][intelpt] Support system-wide tracing [1] - Add a method for accessing the list of logical core ids

Jakob Johnson via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Mon May 2 07:34:59 PDT 2022


jj10306 accepted this revision.
jj10306 added inline comments.


================
Comment at: lldb/unittests/Process/Linux/PerfTests.cpp:53
 
+TEST(Perf, HardcodedLogicalCoreIDs) {
+  Expected<std::vector<int>> core_ids =
----------------
Not directly related to this diff, but I just realized the proc fs parsing logic doesn't belong in the Perf files since it has nothing to do with perf. Feel free to move those functions and these associated tests to a more appropriate location or we can do it in a future diff since this isn't the diff that made that mistake.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D124573



More information about the lldb-commits mailing list