[PATCH] D21675: New ODR checker for modules

Richard Trieu via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Jan 13 11:38:57 PST 2017


rtrieu marked 5 inline comments as done.
rtrieu added a comment.

Comments have been addressed.  I will be testing it for performance impact next.



================
Comment at: include/clang/AST/ODRHash.h:54
+  // in Pointers.
+  size_type NextFreeIndex;
+
----------------
rsmith wrote:
> Is this always the same as `Pointers.size()`?
Yes it is.  It has been removed and Pointers.size() is used now.


https://reviews.llvm.org/D21675





More information about the cfe-commits mailing list