diff options
author | Lang Hames <lhames@gmail.com> | 2010-07-17 06:31:41 +0000 |
---|---|---|
committer | Lang Hames <lhames@gmail.com> | 2010-07-17 06:31:41 +0000 |
commit | d0f6f017319bbc32b57c2e574d774ac91fe20f18 (patch) | |
tree | cabc045feb55ff6743e3cea246eac2cbbcdc6ea8 /lib/CodeGen/PBQP | |
parent | dc1ad22d5351fca6885ba4ecb18967f8ada7d42a (diff) | |
download | llvm-d0f6f017319bbc32b57c2e574d774ac91fe20f18.tar.gz llvm-d0f6f017319bbc32b57c2e574d774ac91fe20f18.tar.bz2 llvm-d0f6f017319bbc32b57c2e574d774ac91fe20f18.tar.xz |
Iterating over sets of pointers in a heuristic was a bad idea. Switching
any command line paramater changed the register allocation produced by
PBQP.
Turns out variety is not the spice of life.
Fixed some comparators, added others. All good now.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@108613 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/CodeGen/PBQP')
-rw-r--r-- | lib/CodeGen/PBQP/Heuristics/Briggs.h | 8 |
1 files changed, 2 insertions, 6 deletions
diff --git a/lib/CodeGen/PBQP/Heuristics/Briggs.h b/lib/CodeGen/PBQP/Heuristics/Briggs.h index 4c1ce119ed..18eaf7c0da 100644 --- a/lib/CodeGen/PBQP/Heuristics/Briggs.h +++ b/lib/CodeGen/PBQP/Heuristics/Briggs.h @@ -52,9 +52,7 @@ namespace PBQP { bool operator()(Graph::NodeItr n1Itr, Graph::NodeItr n2Itr) const { if (s->getSolverDegree(n1Itr) > s->getSolverDegree(n2Itr)) return true; - if (s->getSolverDegree(n1Itr) < s->getSolverDegree(n2Itr)) - return false; - return (&*n1Itr < &*n2Itr); + return false; } private: HeuristicSolverImpl<Briggs> *s; @@ -69,9 +67,7 @@ namespace PBQP { cost2 = g->getNodeCosts(n2Itr)[0] / s->getSolverDegree(n2Itr); if (cost1 < cost2) return true; - if (cost1 > cost2) - return false; - return (&*n1Itr < &*n2Itr); + return false; } private: |