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

_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to