[llvm] [llvm-exegesis] Remove unused Counter::read method (PR #76651)

Aiden Grossman via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 3 11:38:22 PST 2024


================
@@ -356,7 +356,11 @@ class SubProcessFunctionExecutorImpl
       if (ChildExitCode == 0) {
         // The child exited succesfully, read counter values and return
         // success
-        CounterValues[0] = Counter->read();
+        auto CounterValueOrErr = Counter->readOrError();
+        if (!CounterValueOrErr)
+          return CounterValueOrErr.takeError();
+        CounterValues.swap(*CounterValueOrErr);
----------------
boomanaiden154 wrote:

Thanks for pointing this out. Updated in the latest revision.

https://github.com/llvm/llvm-project/pull/76651


More information about the llvm-commits mailing list