[PATCH] D132205: [llvm-tblgen] CodeGenSchedModels::hasReadOfWrite gets wrong predication result

Zixuan Wu via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 19 01:50:12 PDT 2022


zixuan-wu updated this revision to Diff 453929.
zixuan-wu added a comment.

Add testcase.


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

https://reviews.llvm.org/D132205

Files:
  llvm/test/TableGen/CompressWriteLatencyEntry.td
  llvm/utils/TableGen/CodeGenSchedule.cpp


Index: llvm/utils/TableGen/CodeGenSchedule.cpp
===================================================================
--- llvm/utils/TableGen/CodeGenSchedule.cpp
+++ llvm/utils/TableGen/CodeGenSchedule.cpp
@@ -734,14 +734,12 @@
 }
 
 bool CodeGenSchedModels::hasReadOfWrite(Record *WriteDef) const {
-  for (const CodeGenSchedRW &Read : SchedReads) {
-    Record *ReadDef = Read.TheDef;
-    if (!ReadDef || !ReadDef->isSubClassOf("ProcReadAdvance"))
-      continue;
-
-    RecVec ValidWrites = ReadDef->getValueAsListOfDefs("ValidWrites");
-    if (is_contained(ValidWrites, WriteDef)) {
-      return true;
+  for (auto& ProcModel : ProcModels) {
+    const RecVec &RADefs = ProcModel.ReadAdvanceDefs;
+    for (auto& RADef : RADefs) {
+      RecVec ValidWrites = RADef->getValueAsListOfDefs("ValidWrites");
+      if (is_contained(ValidWrites, WriteDef))
+        return true;
     }
   }
   return false;
Index: llvm/test/TableGen/CompressWriteLatencyEntry.td
===================================================================
--- llvm/test/TableGen/CompressWriteLatencyEntry.td
+++ llvm/test/TableGen/CompressWriteLatencyEntry.td
@@ -21,7 +21,8 @@
 
 // CHECK:        MyTargetWriteLatencyTable[] = {
 // CHECK-NEXT:     { 0,  0}, // Invalid
-// CHECK-NEXT:     { 1,  0} // #1 Write_A_Write_B_Write_C
+// CHECK-NEXT:     { 1,  0}, // #1 Write_A_Write_C
+// CHECK-NEXT:     { 1,  2} // #2 Write_B
 // CHECK-NEXT:   }; // MyTargetWriteLatencyTable
 
 let SchedModel = SchedModel_A in {


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D132205.453929.patch
Type: text/x-patch
Size: 1487 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20220819/0c34501e/attachment.bin>


More information about the llvm-commits mailing list