[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