[PATCH] D47166: use zeroinitializer for (trailing zero portion of) large array initializers more reliably

John McCall via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue May 22 11:25:11 PDT 2018


rjmccall added a comment.

I like this approach a lot.



================
Comment at: lib/CodeGen/CGExprConstant.cpp:675
+  // We have mixed types. Use a packed struct.
+  std::vector<llvm::Type *> Types;
+  Types.reserve(Elements.size());
----------------
Why std::vector?


Repository:
  rC Clang

https://reviews.llvm.org/D47166





More information about the cfe-commits mailing list