[prev in list] [next in list] [prev in thread] [next in thread] 

List:       cfe-commits
Subject:    Re: [PATCH] D11298: Convert a few classes over to use the new TrailingObjects helper.
From:       James Y Knight via cfe-commits <cfe-commits () lists ! llvm ! org>
Date:       2015-08-06 20:30:55
Message-ID: ccdbac59bda1e15c322226bf190872af () localhost ! localdomain
[Download RAW message or body]

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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits


[prev in list] [next in list] [prev in thread] [next in thread]