[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