subject:"\[PATCH\] D100120\: \[RISCV\]\[Clang\] Add all RVV Mask intrinsic functions."

[PATCH] D100120: [RISCV][Clang] Add all RVV Mask intrinsic functions.

2021-04-09 Thread Craig Topper via Phabricator via cfe-commits

craig.topper accepted this revision. craig.topper added a comment. This revision is now accepted and ready to land. LGTM Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D100120/new/ https://reviews.llvm.org/D100120 ___

[PATCH] D100120: [RISCV][Clang] Add all RVV Mask intrinsic functions.

2021-04-08 Thread Craig Topper via Phabricator via cfe-commits

craig.topper added a comment. LGTM other than that inheritance question Comment at: clang/include/clang/Basic/riscv_vector.td:247 +list> suffixes_prototypes> { + defm "" : RVVBuiltinSet; +} Can we use inheritance here? Repository: