Skip to content

Commit c802c2d

Browse files
author
Remi Delmas
committed
CONTRACTS: loop assigns clause inference function
1 parent 9a6fbff commit c802c2d

File tree

3 files changed

+134
-0
lines changed

3 files changed

+134
-0
lines changed

src/goto-instrument/Makefile

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -24,6 +24,7 @@ SRC = accelerate/accelerate.cpp \
2424
contracts/dynamic-frames/dfcc_loop_tags.cpp \
2525
contracts/dynamic-frames/dfcc_check_loop_normal_form.cpp \
2626
contracts/dynamic-frames/dfcc_root_object.cpp \
27+
contracts/dynamic-frames/dfcc_infer_loop_assigns.cpp \
2728
contracts/dynamic-frames/dfcc_library.cpp \
2829
contracts/dynamic-frames/dfcc_is_cprover_symbol.cpp \
2930
contracts/dynamic-frames/dfcc_is_fresh.cpp \
Lines changed: 103 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,103 @@
1+
/*******************************************************************\
2+
3+
Module: Dynamic frame condition checking
4+
5+
Author: Remi Delmas, [email protected]
6+
7+
\*******************************************************************/
8+
#include "dfcc_infer_loop_assigns.h"
9+
10+
#include <util/expr.h>
11+
#include <util/find_symbols.h>
12+
#include <util/message.h>
13+
#include <util/pointer_expr.h>
14+
#include <util/std_code.h>
15+
16+
#include <goto-instrument/havoc_utils.h>
17+
#include <langapi/language_util.h>
18+
19+
#include "dfcc_root_object.h"
20+
21+
/// Builds a call expression `object_whole(expr)`
22+
static exprt
23+
make_object_whole_call_expr(const exprt &expr, const namespacet &ns)
24+
{
25+
const symbolt &object_whole_sym = ns.lookup(CPROVER_PREFIX "object_whole");
26+
const code_typet &object_whole_code_type =
27+
to_code_type(object_whole_sym.type);
28+
exprt::operandst args{{expr}};
29+
return side_effect_expr_function_callt(
30+
object_whole_sym.symbol_expr(),
31+
args,
32+
object_whole_code_type.return_type(),
33+
expr.source_location());
34+
}
35+
36+
/// Returns true iff \p expr contains at least one identifier found in
37+
/// \p identifiers.
38+
static bool
39+
depends_on(const exprt &expr, std::unordered_set<irep_idt> identifiers)
40+
{
41+
const std::unordered_set<irep_idt> ids = find_symbol_identifiers(expr);
42+
for(const auto &id : ids)
43+
{
44+
if(identifiers.find(id) != identifiers.end())
45+
return true;
46+
}
47+
return false;
48+
}
49+
50+
std::set<exprt> dfcc_infer_loop_assigns(
51+
const local_may_aliast &local_may_alias,
52+
const loopt &loop_instructions,
53+
const source_locationt &loop_head_location,
54+
messaget &log,
55+
const namespacet &ns)
56+
{
57+
// infer
58+
assignst assigns;
59+
get_assigns(local_may_alias, loop_instructions, assigns);
60+
61+
// compute locals
62+
std::unordered_set<irep_idt> loop_locals;
63+
for(const auto &target : loop_instructions)
64+
{
65+
if(target->is_decl())
66+
loop_locals.insert(target->decl_symbol().get_identifier());
67+
}
68+
69+
// widen or drop targets that depend on loop-locals or are non-constant
70+
havoc_utils_is_constantt is_constant(assigns, ns);
71+
assignst result;
72+
for(const auto &expr : assigns)
73+
{
74+
if(depends_on(expr, loop_locals))
75+
{
76+
// Target depends on loop locals, attempt widening to the root object
77+
auto root_object = dfcc_assigns_target_single_root_object(expr, ns, log);
78+
if(!depends_on(root_object, loop_locals))
79+
{
80+
address_of_exprt address_of_root_object(root_object);
81+
address_of_root_object.add_source_location() =
82+
root_object.source_location();
83+
result.emplace(make_object_whole_call_expr(address_of_root_object, ns));
84+
}
85+
}
86+
else
87+
{
88+
address_of_exprt address_of_expr(expr);
89+
address_of_expr.add_source_location() = expr.source_location();
90+
if(!is_constant(address_of_expr))
91+
{
92+
// Target address is not constant, widening to the whole object
93+
result.emplace(make_object_whole_call_expr(address_of_expr, ns));
94+
}
95+
else
96+
{
97+
result.emplace(expr);
98+
}
99+
}
100+
}
101+
102+
return result;
103+
}
Lines changed: 30 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,30 @@
1+
/*******************************************************************\
2+
3+
Module: Dynamic frame condition checking
4+
5+
Author: Remi Delmas, [email protected]
6+
7+
\*******************************************************************/
8+
9+
/// \file
10+
/// Infer a set of assigns clause targets for a natural loop.
11+
12+
#ifndef CPROVER_GOTO_INSTRUMENT_CONTRACTS_DYNAMIC_FRAMES_DFCC_INFER_LOOP_ASSIGNS_H
13+
#define CPROVER_GOTO_INSTRUMENT_CONTRACTS_DYNAMIC_FRAMES_DFCC_INFER_LOOP_ASSIGNS_H
14+
15+
#include <analyses/local_may_alias.h>
16+
#include <goto-instrument/loop_utils.h>
17+
class source_locationt;
18+
class messaget;
19+
class namespacet;
20+
21+
// \brief Infer assigns clause targets for a loop from its instructions and a
22+
// may alias analysis at the function level.
23+
std::set<exprt> dfcc_infer_loop_assigns(
24+
const local_may_aliast &local_may_alias,
25+
const loopt &loop_instructions,
26+
const source_locationt &loop_head_location,
27+
messaget &log,
28+
const namespacet &ns);
29+
30+
#endif

0 commit comments

Comments
 (0)