[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