Skip to content

Commit

Permalink
Cleanup defunct SIPs
Browse files Browse the repository at this point in the history
These all relied on information about deltas that is no longer available in the
AST.
  • Loading branch information
langston-barrett committed Mar 11, 2022
1 parent 2fdbd07 commit f16d18d
Showing 1 changed file with 0 additions and 36 deletions.
36 changes: 0 additions & 36 deletions src/ast/utility/SipsMetric.h
Original file line number Diff line number Diff line change
Expand Up @@ -100,17 +100,6 @@ class MaxBoundSips : public SipsMetric {
const std::vector<Atom*> atoms, const BindingStore& bindingStore) const override;
};

/** Goal: prioritise (1) all-bound, then (2) max number of bound vars, then (3) left-most, but use deltas as a
* tiebreaker between these. */
class MaxBoundDeltaSips : public SipsMetric {
public:
MaxBoundDeltaSips() = default;

protected:
std::vector<double> evaluateCosts(
const std::vector<Atom*> atoms, const BindingStore& bindingStore) const override;
};

/** Goal: prioritise max ratio of bound args */
class MaxRatioSips : public SipsMetric {
public:
Expand Down Expand Up @@ -158,16 +147,6 @@ class ProfileUseSips : public SipsMetric {
const analysis::ProfileUseAnalysis& profileUse;
};

/** Goal: prioritise (1) all-bound, then (2) deltas, and then (3) left-most */
class DeltaSips : public SipsMetric {
public:
DeltaSips() = default;

protected:
std::vector<double> evaluateCosts(
const std::vector<Atom*> atoms, const BindingStore& bindingStore) const override;
};

/** Goal: prioritise (1) all-bound, then (2) input, and then (3) left-most */
class InputSips : public SipsMetric {
public:
Expand All @@ -183,19 +162,4 @@ class InputSips : public SipsMetric {
const analysis::IOTypeAnalysis& ioTypes;
};

/** Goal: prioritise (1) all-bound, then (2) deltas, then (3) input, and then (4) left-most */
class DeltaInputSips : public SipsMetric {
public:
DeltaInputSips(const Program& program, const analysis::IOTypeAnalysis& ioTypes)
: program(program), ioTypes(ioTypes) {}

protected:
std::vector<double> evaluateCosts(
const std::vector<Atom*> atoms, const BindingStore& bindingStore) const override;

private:
const Program& program;
const analysis::IOTypeAnalysis& ioTypes;
};

} // namespace souffle::ast

0 comments on commit f16d18d

Please sign in to comment.