craig.topper added inline comments.

================
Comment at: clang/include/clang/Basic/BuiltinsRISCV.def:1
+#if defined(BUILTIN) && !defined(RISCVV_BUILTIN)
+#define RISCVV_BUILTIN(ID, TYPE, ATTRS) BUILTIN(ID, TYPE, ATTRS)
----------------
Need copyright header


================
Comment at: clang/include/clang/Basic/DiagnosticSemaKinds.td:11133
+// RISC-V V-extension
+def err_riscvv_builtin_not_useable : Error<
+   "builtin requires 'V' extension support to be enabled">;
----------------
I'm not sure that's the right spelling of useable. It can also be spelled 
usable. But it doesn't appear in the message so maybe the name should be closer 
to wording of the message? Maybe err_riscv_builtin_requires_v. That makes it 
like err_mips_builtin_requires_dsp


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D93446

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to