[PATCH] D68992: [DFAPacketizer] Use DFAEmitter. NFC.

James Molloy via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 15 07:59:22 PDT 2019


jmolloy created this revision.
Herald added subscribers: llvm-commits, hiraditya.
Herald added a project: LLVM.
jmolloy added reviewers: ThomasRaoux, majnemer, kparzysz.

This is a NFC change that removes the NFA->DFA construction and emission logic from DFAPacketizerEmitter and instead uses the generic DFAEmitter logic. This allows DFAPacketizer to use the Automaton class from Support and remove a bunch of logic there too.

After this patch, DFAPacketizer is mostly logic for grepping Itineraries and collecting functional units, with no state machine logic. This will allow us to modernize by removing the 16-functional-unit limit and supporting non-itinerary functional units. This is all for followup patches.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D68992

Files:
  llvm/include/llvm/CodeGen/DFAPacketizer.h
  llvm/include/llvm/Support/Automaton.h
  llvm/lib/CodeGen/DFAPacketizer.cpp
  llvm/test/CodeGen/Hexagon/packetizer-resources.ll
  llvm/utils/TableGen/DFAPacketizerEmitter.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D68992.225043.patch
Type: text/x-patch
Size: 37303 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20191015/c7ad06c2/attachment.bin>


More information about the llvm-commits mailing list