Skip to content

Commit 73af97d

Browse files
committed
Remove interval_abstract_objectt::merge_count
See #5793 (comment)
1 parent fb61df2 commit 73af97d

File tree

2 files changed

+9
-27
lines changed

2 files changed

+9
-27
lines changed

src/analyses/variable-sensitivity/interval_abstract_value.cpp

Lines changed: 9 additions & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -219,33 +219,21 @@ static inline constant_interval_exprt interval_from_relation(const exprt &e)
219219
}
220220

221221
interval_abstract_valuet::interval_abstract_valuet(const typet &t)
222-
: abstract_value_objectt(t), interval(t), merge_count(0)
222+
: abstract_value_objectt(t), interval(t)
223223
{
224224
}
225225

226226
interval_abstract_valuet::interval_abstract_valuet(
227227
const typet &t,
228228
bool tp,
229229
bool bttm)
230-
: abstract_value_objectt(t, tp, bttm), interval(t), merge_count(0)
230+
: abstract_value_objectt(t, tp, bttm), interval(t)
231231
{
232232
}
233233

234234
interval_abstract_valuet::interval_abstract_valuet(
235235
const constant_interval_exprt &e)
236-
: interval_abstract_valuet(e, 0)
237-
{
238-
}
239-
240-
interval_abstract_valuet::interval_abstract_valuet(
241-
const constant_interval_exprt &e,
242-
int merge_count)
243-
: abstract_value_objectt(
244-
e.type(),
245-
e.is_top() || merge_count > 10,
246-
e.is_bottom()),
247-
interval(e),
248-
merge_count(merge_count)
236+
: abstract_value_objectt(e.type(), e.is_top(), e.is_bottom()), interval(e)
249237
{
250238
}
251239

@@ -506,13 +494,11 @@ abstract_object_pointert interval_abstract_valuet::merge_intervals(
506494
}
507495
else
508496
{
509-
return std::make_shared<interval_abstract_valuet>(
510-
constant_interval_exprt(
511-
constant_interval_exprt::get_min(
512-
interval.get_lower(), other->interval.get_lower()),
513-
constant_interval_exprt::get_max(
514-
interval.get_upper(), other->interval.get_upper())),
515-
std::max(merge_count, other->merge_count) + 1);
497+
return std::make_shared<interval_abstract_valuet>(constant_interval_exprt(
498+
constant_interval_exprt::get_min(
499+
interval.get_lower(), other->interval.get_lower()),
500+
constant_interval_exprt::get_max(
501+
interval.get_upper(), other->interval.get_upper())));
516502
}
517503
}
518504

@@ -559,8 +545,7 @@ abstract_object_pointert interval_abstract_valuet::meet_intervals(
559545
return std::make_shared<interval_abstract_valuet>(
560546
interval.type(), false, true);
561547
return std::make_shared<interval_abstract_valuet>(
562-
constant_interval_exprt(lower_bound, upper_bound),
563-
std::max(merge_count, other->merge_count) + 1);
548+
constant_interval_exprt(lower_bound, upper_bound));
564549
}
565550
}
566551

src/analyses/variable-sensitivity/interval_abstract_value.h

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,6 @@ class interval_abstract_valuet : public abstract_value_objectt
3131
interval_abstract_valuet(const typet &t, bool tp, bool bttm);
3232

3333
explicit interval_abstract_valuet(const constant_interval_exprt &e);
34-
interval_abstract_valuet(const constant_interval_exprt &e, int merge_count);
3534

3635
interval_abstract_valuet(
3736
const exprt &e,
@@ -101,8 +100,6 @@ class interval_abstract_valuet : public abstract_value_objectt
101100
meet_intervals(interval_abstract_value_pointert other) const;
102101

103102
constant_interval_exprt interval;
104-
105-
int merge_count;
106103
};
107104

108105
#endif // CPROVER_ANALYSES_VARIABLE_SENSITIVITY_INTERVAL_ABSTRACT_VALUE_H

0 commit comments

Comments
 (0)