[PATCH] Add standard insert overloads to StringMap

Agustín Bergé kaballo86 at hotmail.com
Mon Jun 16 18:08:55 PDT 2014


Changes in this iteration:

- Removed extra variable in `insert`.
- Removed unwanted changes from clang-format.
- Added a test case (attempt) for insertion where rehashing happens, and the bucket pointing to the element changes. This is a bad test, but I'm including it as a base for discussion on how should such a test look, if it's even sensible to have it.

http://reviews.llvm.org/D4153

Files:
  include/llvm/ADT/StringMap.h
  lib/Support/StringMap.cpp
  unittests/ADT/StringMapTest.cpp
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D4153.10473.patch
Type: text/x-patch
Size: 5825 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140617/6b9f05b4/attachment.bin>


More information about the llvm-commits mailing list