[all-commits] [llvm/llvm-project] 617b08: Refactor StringMap.h, splitting StringMapEntry out...

Chris Lattner via All-commits all-commits at lists.llvm.org
Sun Apr 12 08:25:35 PDT 2020


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: 617b08ff9bef7b17957781682f78155351b0d2e9
      https://github.com/llvm/llvm-project/commit/617b08ff9bef7b17957781682f78155351b0d2e9
  Author: Chris Lattner <clattner at nondot.org>
  Date:   2020-04-12 (Sun, 12 Apr 2020)

  Changed paths:
    M llvm/include/llvm/ADT/StringMap.h
    A llvm/include/llvm/ADT/StringMapEntry.h
    M llvm/lib/Support/StringMap.cpp

  Log Message:
  -----------
  Refactor StringMap.h, splitting StringMapEntry out to its own header.

Summary:
StringMapEntry.h can have lower dependencies, than StringMap.h, which
is useful for public headers that want to expose inline methods on
StringMapEntry<> but don't need to expose all of StringMap.h.  One
example of this is mlir's Identifier.h, another example is the existing
LLVM StringPool.h.

StringPool also could use a cleanup, I'll deal with that in a follow-on
patch.

Reviewers: rriddle

Subscribers: hiraditya, dexonsmith, llvm-commits

Tags: #llvm

Differential Revision: https://reviews.llvm.org/D77963




More information about the All-commits mailing list