[PATCH] D121586: [Clang][VE] Add the rest of intrinsics to clang

[Kazushi Marukawa via Phabricator via cfe-commits]

kaz7 added inline comments.


================
Comment at: clang/include/clang/Basic/BuiltinsVEVL.gen.def:35
+// TODO: Vector mask registers
+// Depend on https://reviews.llvm.org/D88905
+// BUILTIN(__builtin_ve_vl_vst_vssml, "vV256dLUiv*V256bUi", "n")
----------------
xbolva00 wrote:
> Why leave dead code under comments? Bad habit, remove it please.
Thank you for the comments.  What is the best way to leave a list of functions need to be supported in near future?  Should I just make a list of function names using vector mask registers instead of writing them down with "BUILTIN" for future reference?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D121586