[PATCH] D91927: [X86] Add x86_amx type for intel AMX.
Pengfei Wang via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Dec 30 06:35:35 PST 2020
pengfei added inline comments.
================
Comment at: llvm/include/llvm/IR/Type.h:68
X86_MMXTyID, ///< MMX vectors (64 bits, X86 specific)
+ X86_AMXTyID, ///< AMX vectors (8192 bits, X86 specific)
TokenTyID, ///< Tokens
----------------
uabelho wrote:
> This addition causes a compilation warning in HexagonTargetObjectFile.cpp:
>
> ```
> ../lib/Target/Hexagon/HexagonTargetObjectFile.cpp:297:11: error: enumeration value 'X86_AMXTyID' not handled in switch [-Werror,-Wswitch]
> switch (Ty->getTypeID()) {
> ^
> 1 error generated.
> ```
> Seen in build bots, e.g. here:
> http://lab.llvm.org:8011/#/builders/57/builds/2889/steps/6/logs/stdio
Thanks Mikael for pointing it out. I think we just need to put the type in the switch table. I've posted a patch to fix it. rG16c2067cf212.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D91927/new/
https://reviews.llvm.org/D91927
More information about the cfe-commits
mailing list