diff options
author | Duncan P. N. Exon Smith <dexonsmith@apple.com> | 2016-08-17 02:08:08 +0000 |
---|---|---|
committer | Duncan P. N. Exon Smith <dexonsmith@apple.com> | 2016-08-17 02:08:08 +0000 |
commit | 47416615737795327efc3ce3c99566debe16ed04 (patch) | |
tree | 19bb036a2c2aad099f10fea0a953619e863aad04 /llvm/lib/Object/SymbolicFile.cpp | |
parent | 362d120488722bf91824f8e74748459525d10614 (diff) | |
download | llvm-47416615737795327efc3ce3c99566debe16ed04.zip llvm-47416615737795327efc3ce3c99566debe16ed04.tar.gz llvm-47416615737795327efc3ce3c99566debe16ed04.tar.bz2 |
ADT: Add some missing coverage for iplist::splice
These splices are interesting because they involve swapping two nodes in
the same list. There are two ways to do this. Assuming:
A -> B -> [Sentinel]
You can either:
- splice B before A, with: L.splice(A, L, B) or
- splice A before Sentinel, with: L.splice(L.end(), L, A) to create:
B -> A -> [Sentinel]
These two swapping-splices are somewhat interesting corner cases for
maintaining the list invariants. The tests pass even with my new ilist
implementation, but I had some doubts about the latter when I was
looking at weird UB effects. Since I can't find equivalent explicit
test coverage elsewhere it seems prudent to commit.
llvm-svn: 278887
Diffstat (limited to 'llvm/lib/Object/SymbolicFile.cpp')
0 files changed, 0 insertions, 0 deletions