[PATCH] D11298: Convert a few classes over to use the new TrailingObjects helper.

James Y Knight via cfe-commits cfe-commits at lists.llvm.org
Thu Aug 6 13:30:55 PDT 2015


jyknight marked 2 inline comments as done.

================
Comment at: lib/AST/Decl.cpp:3122
@@ +3121,3 @@
+  void *Buffer = Context.Allocate(
+      totalSizeToAlloc<TemplateArgumentLoc, FunctionTemplateDecl *>(
+          TArgs.size(), Ts.size()));
----------------
rsmith wrote:
> OK, I guessed that might be the reason. Seems like a reasonable safety measure. (Do you get an error if you specify the wrong types?)
Yep, the function has an enable_if to ensure you call it with the right types.


Repository:
  rL LLVM

http://reviews.llvm.org/D11298





More information about the cfe-commits mailing list