[PATCH] D44309: [llvm-mca] Refactor event listeners to make the backend agnostic to event types.

Clement Courbet via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 13 05:22:23 PDT 2018


courbet added a comment.

Thanks !



================
Comment at: tools/llvm-mca/TimelineView.cpp:44
+    TVEntry.CycleRetired = CurrentCycle;
+    LastCycle = std::max(LastCycle, CurrentCycle);
+
----------------
RKSimon wrote:
> Isn't this repeated at the end of the function?
Indeed, thanks for the catch.


Repository:
  rL LLVM

https://reviews.llvm.org/D44309





More information about the llvm-commits mailing list