[PATCH] D143364: [RISCV] Support scalar/fix-length vector NTLH intrinsic with different domain

[Craig Topper via Phabricator via cfe-commits]

craig.topper added inline comments.


================
Comment at: clang/test/CodeGen/RISCV/ntlh-intrinsics/riscv32-zihintntl.c:19
+typedef signed short v8ss __attribute__((vector_size(16)));
+typedef signed char v16sc __attribute__((vector_size(16)));
+v4si v4si1, v4si2;
----------------
BeMg wrote:
> craig.topper wrote:
> > What about the rvv builtin types for scalable vectors?
> Should we support rvv builtin types for __riscv_ntl_load/store although we already have plan to support RVV builtin type in nontemporal by extra RVV intrinsic functions?
> 
> It need some extra code to support these function with scalable vector type. 
Can you add tests to show it is not supported and gives an error message?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D143364