[Lldb-commits] [PATCH] D123984: [trace][intel pt] Add a memory usage test

[Jakob Johnson via Phabricator via lldb-commits]

jj10306 added a comment.

What is the intention of this test? To catch changes that may increase the memory by a lot? I'm concerned this could be very flaky given that it's dependent on external factors (compiler output, context switches, etc).
Now that we have this test, is there any value in having the check for the memory information output in the trace dump test or can that be removed?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D123984