[PATCH] D77976: NFC: Clean up the implementation of StringPool a bit, and remove dependence on some "implicitly MallocAllocator" based methods on StringMapEntry. This allows reducing the #includes in StringMapEntry.h.

Chris Lattner via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Apr 12 11:12:53 PDT 2020


lattner created this revision.
lattner added a reviewer: rriddle.
Herald added subscribers: llvm-commits, dexonsmith, hiraditya.
Herald added a project: LLVM.

StringPool has many caveats and isn't used in the monorepo.  I will
propose removing it as a patch separate from this refactoring patch.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D77976

Files:
  llvm/include/llvm/ADT/StringMap.h
  llvm/include/llvm/ADT/StringMapEntry.h
  llvm/include/llvm/Support/StringPool.h
  llvm/lib/Support/StringPool.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D77976.256866.patch
Type: text/x-patch
Size: 10533 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200412/1da58d15/attachment.bin>


More information about the llvm-commits mailing list