diff options
author | Manuel Klimek <klimek@google.com> | 2013-02-13 10:46:36 +0000 |
---|---|---|
committer | Manuel Klimek <klimek@google.com> | 2013-02-13 10:46:36 +0000 |
commit | 32a2fd7631026f2fe248381546c5e6149f4f95ee (patch) | |
tree | 5fd3bdb030d9168fd9359c5ae9c293665bc77ef7 /lib/CodeGen/CGClass.cpp | |
parent | 59660c21178b6af518bd4b564e032d5c9cc218cb (diff) |
An attempt to make the search algorithm easier to understand.
- clear ownership: the SpecificBumpPtrAllocator owns all StateNodes
- this allows us to simplify the memoization data structure into a
std::set (FIXME: figure out whether we want to use a hash based
data structure).
- introduces StateNode as recursive data structure, instead of using
Edge and the Seen-map combined to drill through the graph
- using a count to stabilize the penalty instead of relying on the
container
- pulled out a method to forward-apply states in the end
This leads to a ~40% runtime decrease on Nico's benchmark.
Main FiXME is that the parameter lists of some function get too long.
I'd vote for either pulling the Queue etc into the Formatter proper,
or creating an inner class just for the search algorithm.
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@175051 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/CodeGen/CGClass.cpp')
0 files changed, 0 insertions, 0 deletions