[PATCH] D143511: [CodeGen] Add standard print/debug utilities to MVT

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 7 10:38:42 PST 2023


craig.topper accepted this revision.
craig.topper added a comment.
This revision is now accepted and ready to land.

LGTM



================
Comment at: llvm/lib/CodeGen/ValueTypes.cpp:620
+#if !defined(NDEBUG) || defined(LLVM_ENABLE_DUMP)
+void MVT::dump() const {
+  print(dbgs());
----------------
reames wrote:
> craig.topper wrote:
> > Is this a library layering issue?
> Yes, but not a new one.  MVT::getVT is defined a couple lines above, so if that links, so should this.  
I suppose it means that dump wouldn't work from gdb for tblgen, but would for other tools?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D143511



More information about the llvm-commits mailing list