[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