[PATCH] D56455: [ADT] Fix SmallDenseMap assertion with large InlineBuckets

Andrew Trick via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 9 18:45:26 PDT 2019


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

Hmm... I guess the intent of the original design is not to overload the table even if it's in small mode. I'm not sure that makes sense, but I'll accept it as status quo. Anyway, it does sort of make sense that we can compress the small table by removing tombstones.

Now that I fully understand what's happening, this looks like a good fix!


Repository:
  rL LLVM

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

https://reviews.llvm.org/D56455





More information about the llvm-commits mailing list