llvm/unittests
Chandler Carruth 6253c04fc9 [LCG] Actually test the *basic* edge removal bits (IE, the non-SCC
bits), and discover that it's totally broken. Yay tests. Boo bug. Fix
the basic edge removal so that it works by nulling out the removed edges
rather than actually removing them. This leaves the indices valid in the
map from callee to index, and preserves some of the locality for
iterating over edges. The iterator is made bidirectional to reflect that
it now has to skip over null entries, and the skipping logic is layered
onto it.

As future work, I would like to track essentially the "load factor" of
the edge list, and when it falls below a threshold do a compaction.

An alternative I considered (and continue to consider) is storing the
callees in a doubly linked list where each element of the list is in
a set (which is essentially the classical linked-hash-table
datastructure). The problem with that approach is that either you need
to heap allocate the linked list nodes and use pointers to them, or use
a bucket hash table (with even *more* linked list pointer overhead!),
etc. It's pretty easy to get 5x overhead for values that are just
pointers. So far, I think punching holes in the vector, and periodic
compaction is likely to be much more efficient overall in the space/time
tradeoff.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@207619 91177308-0d34-0410-b5e6-96231b3b80d8
2014-04-30 07:45:27 +00:00
..
ADT [ADT] Teach PointerUnion to support assignment directly from nullptr to 2014-04-29 00:14:27 +00:00
Analysis [LCG] Actually test the *basic* edge removal bits (IE, the non-SCC 2014-04-30 07:45:27 +00:00
Bitcode verify-di: Implement DebugInfoVerifier 2014-04-15 16:27:38 +00:00
CodeGen DIE: Pass ownership of children via std::unique_ptr rather than raw pointer. 2014-04-25 20:00:34 +00:00
DebugInfo [CMake] Update LLVM_LINK_COMPONENTS for each CMakeLists.txt. 2013-12-10 11:13:32 +00:00
ExecutionEngine [PM] Add pass run listeners to the pass manager. 2014-04-28 18:19:25 +00:00
IR Make the User::value_op_iterator a random access iterator. I had written 2014-04-17 09:07:50 +00:00
LineEditor raw_ostream: Forward declare OpenFlags and include FileSystem.h only where necessary. 2014-04-29 23:26:49 +00:00
Linker LinkModulesTest.cpp: Reformat. 2014-04-29 15:52:46 +00:00
MC Fix MCDataAtom never calling remap when adding data. 2013-10-16 18:26:16 +00:00
Object Object: Don't double-escape empty hexdata 2014-03-20 06:28:52 +00:00
Option Retire llvm::array_endof in favor of non-member std::end. 2014-04-12 16:15:53 +00:00
Support SupportTest: Fix test names harder 2014-04-29 17:07:45 +00:00
Transforms This reverts r206828 until David has time to figure out that is going on. 2014-04-22 02:17:11 +00:00
CMakeLists.txt [CMake] Enable llvm/unittests/LinkerTests. It had not been enabled since r199354. 2014-04-29 15:52:36 +00:00
Makefile Introduce line editor library. 2014-01-31 23:46:14 +00:00
Makefile.unittest With rpaths being set correctly, SHLIBPATH_VAR is not needed anymore. 2014-02-28 16:16:51 +00:00