[PATCH] D76690: [AST][SVE] Treat built-in SVE types as POD

Eli Friedman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Mar 24 12:54:07 PDT 2020


efriedma added inline comments.


================
Comment at: clang/lib/AST/Type.cpp:2515
+  if (BaseTy->isSizelessBuiltinType())
+    return true;
+
----------------
Can you rearrange this so isSizelessBuiltinType() is at the bottom?  It should be rare.  (Assuming it doesn't need to be before the isIncompleteType() check.)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D76690





More information about the cfe-commits mailing list