[PATCH] D47637: Check Sched Class tables at generation time

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 18 06:08:58 PDT 2018


RKSimon added reviewers: courbet, andreadb, gchatelet, lebedev.ri.
RKSimon added a subscriber: llvm-commits.
RKSimon added inline comments.


================
Comment at: utils/TableGen/SubtargetEmitter.cpp:1285
+  // the same schedule as its default class.
+  for (const CodeGenInstruction *Inst : TGT.getInstructionsByEnumValue()) {
+    const std::pair<unsigned, unsigned> ScIdx =
----------------
I think this patch should be limited to just this warning and remove the other check loops below for future patches - after we've at least cleaned up these 'unnecessary overrides' from the scheduler models.


https://reviews.llvm.org/D47637





More information about the llvm-commits mailing list