diff options
author | Justin Lebar <jlebar@google.com> | 2016-11-05 16:47:25 +0000 |
---|---|---|
committer | Justin Lebar <jlebar@google.com> | 2016-11-05 16:47:25 +0000 |
commit | 54b0be048e3e449aadf7206efa991ee3419188ac (patch) | |
tree | 97b913f7630436a86dfe534c5982f375993becd3 /clang/unittests/Format/FormatTest.cpp | |
parent | 1af0fc1103591c35cca8048dae9776a6c329ecd1 (diff) | |
download | bcm5719-llvm-54b0be048e3e449aadf7206efa991ee3419188ac.tar.gz bcm5719-llvm-54b0be048e3e449aadf7206efa991ee3419188ac.zip |
[LoopStrengthReduce] Don't use a DenseSet<int64_t> when we might add any valid int64_t to the set.
Summary:
SmallSetVector uses DenseSet, but that means we need to reserve some
values for the empty and tombstone keys.
It seems to me we should have a general way to let us store full-range
ints inside of DenseSets, and furthermore that we probably shouldn't
silently let you add ints into DenseSets without explicitly promising
that they're in range. But that's a battle for another day; for now,
just fix this code, since we currently do something Very Bad when
compiling ffmpeg.
Fixes PR30914.
Reviewers: jeremyhu
Subscribers: llvm-commits, mzolotukhin
Differential Revision: https://reviews.llvm.org/D26323
llvm-svn: 286038
Diffstat (limited to 'clang/unittests/Format/FormatTest.cpp')
0 files changed, 0 insertions, 0 deletions