[PATCH] D72293: [mlir] NFC: Move the state for managing SSA value names out of OperationPrinter and into a new class SSANameState.

River Riddle via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 6 10:44:41 PST 2020


rriddle created this revision.
rriddle added a reviewer: jpienaar.
Herald added subscribers: llvm-commits, lucyrfox, mgester, arpith-jacob, nicolasvasilache, antiagainst, shauheen, burmako, mehdi_amini.
Herald added a project: LLVM.
rriddle added a child revision: D72294: [mlir] Refactor ModuleState into AsmState and expose it to users..

This reduces the complexity of OperationPrinter and simplifies the code by quite a bit. The SSANameState is now held by ModuleState. This is in preparation for a future revision that molds ModuleState into something that can be used by users for caching the printer state, as well as for implementing printAsOperand style methods.

Depends On D72292 <https://reviews.llvm.org/D72292>


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D72293

Files:
  mlir/lib/IR/AsmPrinter.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D72293.236415.patch
Type: text/x-patch
Size: 60280 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200106/4e46d22d/attachment.bin>


More information about the llvm-commits mailing list