diff options
| author | Chandler Carruth <chandlerc@gmail.com> | 2016-12-06 10:29:23 +0000 |
|---|---|---|
| committer | Chandler Carruth <chandlerc@gmail.com> | 2016-12-06 10:29:23 +0000 |
| commit | 23a6c3f7462ad79be7b135e8adf246bbc756a1cb (patch) | |
| tree | b8eaa70296a95f19d9bb6acd09898a84d1eb6e17 /lldb/source/Commands/CommandObjectExpression.cpp | |
| parent | 7582c669bdc4aed10c18eb799313c2b6f7e71edb (diff) | |
| download | llvm-23a6c3f7462ad79be7b135e8adf246bbc756a1cb.zip llvm-23a6c3f7462ad79be7b135e8adf246bbc756a1cb.tar.gz llvm-23a6c3f7462ad79be7b135e8adf246bbc756a1cb.tar.bz2 | |
[LCG] Add some much needed asserts and verify runs to uncover
a hilarious bug and fix it.
We somehow were never verifying the RefSCCs newly formed when
splitting an existing one apart, and when verifying them we weren't
really checking the SCC indices mapping effectively.
If we had been, it would have been blindingly obvious that right after
putting something int `RC.SCCs` we should update `RC.SCCIndices` instead
of `SCCIndices` which we were about to clear and rebuild anyways. =[
Anyways, this is thoroughly covered by existing tests now that we
actually verify things properly.
llvm-svn: 288795
Diffstat (limited to 'lldb/source/Commands/CommandObjectExpression.cpp')
0 files changed, 0 insertions, 0 deletions