[PATCH] D68586: Save a word in every StringSet entry

David Blaikie via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 10 12:00:01 PDT 2019


dblaikie accepted this revision.
dblaikie added a comment.
This revision is now accepted and ready to land.

Looks good to me - thanks!

If you like, maybe leave a comment/hint that this could be generalized to a full EBO if someone has a need in the future.

Any idea why MDString is friending an implementation detail like this? Should it be? Could we make it an actual private implementation detail so people can't do this?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D68586





More information about the llvm-commits mailing list