[llvm] [TableGen][SubtargetEmitter] Refactor hasReadOfWrite to take ProcModel argument (PR #92032)

Craig Topper via llvm-commits llvm-commits at lists.llvm.org
Mon May 13 15:46:06 PDT 2024


================
@@ -1122,8 +1122,8 @@ void SubtargetEmitter::GenSchedClassTables(const CodeGenProcModel &ProcModel,
       WriterNames.push_back(SchedModels.getSchedWrite(WriteID).Name);
       // If this Write is not referenced by a ReadAdvance, don't distinguish it
       // from other WriteLatency entries.
-      if (!SchedModels.hasReadOfWrite(
-              SchedModels.getSchedWrite(WriteID).TheDef)) {
+      if (!SchedModels.hasReadOfWrite(SchedModels.getSchedWrite(WriteID).TheDef,
+                                      ProcModel)) {
----------------
topperc wrote:

The old code here checked all ProcModels not just this ProcModel. I'm surprised that this doesn't change the output.

https://github.com/llvm/llvm-project/pull/92032


More information about the llvm-commits mailing list