[PATCH] D81816: [PowerPC] Add support for vector bool __int128 for Power10

Ahsan Saghir via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Jun 17 13:29:52 PDT 2020


saghir marked 2 inline comments as done.
saghir added inline comments.


================
Comment at: clang/lib/Sema/DeclSpec.cpp:1170
 
       // Only 'short' and 'long long' are valid with vector bool. (PIM 2.1)
       if ((TypeSpecWidth != TSW_unspecified) && (TypeSpecWidth != TSW_short) &&
----------------
lei wrote:
> Do we not need to add a check for int128 to this check as well? 
No need here because this checks for type `TST_int` only.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D81816





More information about the cfe-commits mailing list