[PATCH] D90592: [llvm-exegesis] Save target state before running the benchmark.

Guillaume Chatelet via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 2 04:26:23 PST 2020


gchatelet added a comment.

Can you reformat to fix Lint errors as well.



================
Comment at: llvm/tools/llvm-exegesis/lib/Target.h:178
+  // By default no state is preserved.
+  struct PreservedState {
+    virtual ~PreservedState();
----------------
I find `PreservedState` a bit unclear. Maybe `SavedState` or `SnapshotState` is more explicit? No strong opinion though.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D90592



More information about the llvm-commits mailing list