[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