[PATCH] D89114: [TableGen][SchedModels] Fix aliasing of SchedWriteVariant

Eugene Leviant via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 9 03:43:22 PDT 2020


evgeny777 created this revision.
evgeny777 added reviewers: atrick, javed.absar, craig.topper, andreadb, dmgreen.
Herald added a subscriber: pengfei.
Herald added a project: LLVM.
evgeny777 requested review of this revision.

Atm this doesn't work:

  def MyWriteV : SchedWriteVariant<...>;
  def : SchedAlias<WriteV, MyWriteV>;

One problem with statements above is that assertion is triggered when `inferFromTransitions` tries to add empty write sequence which triggers assertion in findOrInsertRW.
Another problem is that transition is always added for given scheduling class, even though that class already has InstRW for a same processor.
Patch attempts to fix both of them


https://reviews.llvm.org/D89114

Files:
  llvm/test/TableGen/sched-aliases.td
  llvm/utils/TableGen/CodeGenSchedule.cpp
  llvm/utils/TableGen/CodeGenSchedule.h

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D89114.297176.patch
Type: text/x-patch
Size: 7303 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20201009/6a956ce8/attachment.bin>


More information about the llvm-commits mailing list