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

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
  • [PATCH] D114425: ... Aaron Ballman via Phabricator via cfe-commits

Reply via email to