[PATCH] D25645: [ADT] Add CachedHashString.

[Mehdi AMINI via llvm-commits]

mehdi_amini added a comment.

Couldn't `CachedHashString` just wrap a `CachedHashStringRef`?

Or even have a single implementation:

  template<bool Owning>
  class CachedHashStringRefBase {
    ...
  
    ~CachedHashStringRefBase() {
      if (owning) delete...
  }
  using CachedHashStringRef = CachedHashStringRefBase<false>;
  using CachedHashString = CachedHashStringRefBase<true>;




https://reviews.llvm.org/D25645