[PATCH] D74589: [ADT] Allow empty string in StringSet

Sam Clegg via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 27 16:00:05 PDT 2020


sbc100 added a comment.

Added tests.

>From inspection I can't see why this limitation would be needed.   I also looked back though the history to identify the original reason for this limitation and it came, not from the data structure but from its domain-specific origin (a place where empty strings were not expected).


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D74589/new/

https://reviews.llvm.org/D74589





More information about the llvm-commits mailing list