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

Philip Reames via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 8 07:28:44 PST 2023


reames added a comment.

In D143511#4112081 <https://reviews.llvm.org/D143511#4112081>, @sarahpurohit wrote:

> What about the use of this in TableGen?
>
> It doesn't link to CodeGen (which has the EVT::getEVTString implementation) due to layering constraints, as far as I know, anyway.

First, let me confirm.  You are *not* seeing a build failure right?  I've gotten no builbot failures here which look plausibly connected, but just checking.

Second, what usage in tablegen?  Do you mean in source?  If so, I can't find any such.  If you mean from GDB, then no, it wouldn't work.  If you have a suggestion on a better place for this, please let me know.  It needs to be somewhere I can move the existing MVT:: definitions as well.  I followed the existing precedent for MVT definitions here.


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