[PATCH] D47942: [SmallSet] Add SmallSetIterator.

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 15 01:05:46 PDT 2018


fhahn updated this revision to Diff 151463.
fhahn added a comment.

Replace unneeded destructor with static assertion making sure everything required by the rule of five is trivial. I also updated the test as suggested and added another test with a type with non-trivial ctors/dtor.


https://reviews.llvm.org/D47942

Files:
  include/llvm/ADT/SmallSet.h
  unittests/ADT/SmallSetTest.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D47942.151463.patch
Type: text/x-patch
Size: 5449 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180615/0be5bbd9/attachment.bin>


More information about the llvm-commits mailing list