[PATCH] D108443: [MCA] Fixing bug causing LSUnit to not be notified when a 0 latency instruction finishes executing.

Patrick Holland via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 20 10:34:39 PDT 2021


This revision was automatically updated to reflect the committed changes.
Closed by commit rG3b3c01348be0: [MCA] Fixing bug that was causing LSUnit not to realize an instruction finished… (authored by holland11).

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D108443/new/

https://reviews.llvm.org/D108443

Files:
  llvm/lib/MCA/Stages/InOrderIssueStage.cpp


Index: llvm/lib/MCA/Stages/InOrderIssueStage.cpp
===================================================================
--- llvm/lib/MCA/Stages/InOrderIssueStage.cpp
+++ llvm/lib/MCA/Stages/InOrderIssueStage.cpp
@@ -261,6 +261,7 @@
   // the execution and retirement now.
   if (IS.isExecuted()) {
     PRF.onInstructionExecuted(&IS);
+    LSU.onInstructionExecuted(IR);
     notifyEvent<HWInstructionEvent>(
         HWInstructionEvent(HWInstructionEvent::Executed, IR));
     LLVM_DEBUG(dbgs() << "[E] Instruction #" << IR << " is executed\n");


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D108443.367830.patch
Type: text/x-patch
Size: 544 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210820/8a1fab49/attachment-0001.bin>


More information about the llvm-commits mailing list