[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