[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