[PATCH] D88905: [Clang] Allow "ext_vector_type" applied to Booleans

[Kazushi Marukawa via Phabricator via cfe-commits]

kaz7 added a comment.

We are using this patch in llvm/clang for VE and the patch is really helpful to implement intrinsic instructions using vector mask registers.  This works fine in backend since backend supports vXi1.  I wish people working on clang think this patch is reasonable.



================
Comment at: clang/docs/LanguageExtensions.rst:466
+  typedef bool bool4 __attribute__((ext_vector_type(4)));
+  // Objects of bool8 type hold 8 bits, sizeof(bool8) == 1
+
----------------
rsmith wrote:
> Comment talks about `bool8` but we defined the type `bool4`.
This is not fixed yet, `sizeof(bool8)`.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D88905