[PATCH] D114425: [clang] Add __builtin_bswap128

[Aaron Ballman via Phabricator via cfe-commits]

aaron.ballman added inline comments.


================
Comment at: clang/lib/AST/ASTContext.cpp:11071-11072
 
-    QualType ElementType = DecodeTypeFromStr(Str, Context, Error,
-                                             RequiresICE, false);
+    QualType ElementType =
+        DecodeTypeFromStr(Str, Context, Error, RequiresICE, false, true);
     assert(!RequiresICE && "Can't require vector ICE");
----------------
Why is this change needed? We don't seem to make a vector of __int128 as part of this patch, so I thought we wouldn't need the extra `AllowInt128` parameter to the function.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D114425