[PATCH] D103205: [MIRPrinter] Add machine metadata support.

Michael Liao via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu May 27 11:50:06 PDT 2021


hliao updated this revision to Diff 348344.
hliao added a comment.

Revise that machine tracker a bit to ensure machine metadata is alway per-function state.

- The printer of the machine function is done on a per-function wise. That makes the scanning of machine data through module unnecessary and problematic as MF may be removed deliberately under certain cases. I found that issue when validating the parser support.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D103205/new/

https://reviews.llvm.org/D103205

Files:
  llvm/include/llvm/CodeGen/MIRYamlMapping.h
  llvm/include/llvm/CodeGen/MachineModuleSlotTracker.h
  llvm/include/llvm/IR/ModuleSlotTracker.h
  llvm/lib/CodeGen/CMakeLists.txt
  llvm/lib/CodeGen/MIRPrinter.cpp
  llvm/lib/CodeGen/MachineModuleSlotTracker.cpp
  llvm/lib/IR/AsmWriter.cpp
  llvm/unittests/CodeGen/CMakeLists.txt
  llvm/unittests/CodeGen/MachineMetadata.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D103205.348344.patch
Type: text/x-patch
Size: 33767 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210527/9cfe4caa/attachment.bin>


More information about the llvm-commits mailing list