[PATCH] D44741: [llvm-mca] Move the logic that computes the register file usage to the BackendStatistics view.

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 21 09:58:39 PDT 2018


RKSimon added inline comments.


================
Comment at: tools/llvm-mca/BackendStatistics.cpp:28
+  case HWInstructionEvent::Retired: {
+    const HWInstructionRetiredEvent &RetiredEvent =
+        static_cast<const HWInstructionRetiredEvent &>(Event);
----------------
const auto for casts - we already know the type


================
Comment at: tools/llvm-mca/BackendStatistics.cpp:41
+    const HWInstructionDispatchedEvent &RetiredEvent =
+        static_cast<const HWInstructionDispatchedEvent &>(Event);
+    for (unsigned I = 0, E = RegisterFiles.size(); I < E; ++I) {
----------------
const auto for casts - we already know the type


https://reviews.llvm.org/D44741





More information about the llvm-commits mailing list