[PATCH] D81724: [MVT] Add new MVT types for RISC-V vector.

Hsiangkai Wang via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 12 08:38:06 PDT 2020


HsiangKai marked an inline comment as done.
HsiangKai added inline comments.


================
Comment at: llvm/utils/TableGen/IntrinsicEmitter.cpp:250
+  IIT_V256 = 48,
+  IIT_BF16 = 49
 };
----------------
rogfer01 wrote:
> I wonder if we better preserve the existing numbering here, just in case these numbers have got already been serialized somewhere.
> 
> That would be my impression given the unusual ordering of some of the existing enumerators like `IIT_V512` and `IIT_V128`.
Good point. I didn't aware of this.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D81724





More information about the llvm-commits mailing list