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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
