[PATCH] D31171: Improve StringMap iterator support.

Zachary Turner via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 21 10:13:25 PDT 2017


zturner updated this revision to Diff 92507.
zturner added a comment.

1. Fixed issue with returning by value from `operator*`.
2. Moved the test from Support to ADT, since there was already an existing `StringMapTest` class there that I hadn't noticed.
3. Removed the `join` changes that had snuck in.  I'll commit those separately.


https://reviews.llvm.org/D31171

Files:
  llvm/include/llvm/ADT/STLExtras.h
  llvm/include/llvm/ADT/StringMap.h
  llvm/unittests/ADT/StringMapTest.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D31171.92507.patch
Type: text/x-patch
Size: 7318 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170321/1b0aa544/attachment.bin>


More information about the llvm-commits mailing list