Skip to content

Commit 025ff40

Browse files
author
Remi Delmas
committed
CONTRACTS: loop assigns clause inference function
1 parent e937ead commit 025ff40

File tree

2 files changed

+131
-0
lines changed

2 files changed

+131
-0
lines changed
Lines changed: 101 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,101 @@
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+
code_typet object_whole_code_type = to_code_type(object_whole_sym.type);
27+
exprt::operandst args;
28+
args.emplace_back(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+
static bool
37+
depends_on(const exprt &expr, std::unordered_set<irep_idt> identifiers)
38+
{
39+
const std::unordered_set<irep_idt> ids = find_symbol_identifiers(expr);
40+
for(const auto &id : ids)
41+
{
42+
if(identifiers.find(id) != identifiers.end())
43+
return true;
44+
}
45+
return false;
46+
}
47+
48+
std::set<exprt> dfcc_infer_loop_assigns(
49+
const local_may_aliast &local_may_alias,
50+
const loopt &loop_instructions,
51+
const source_locationt &loop_head_location,
52+
messaget &log,
53+
const namespacet &ns)
54+
{
55+
// infer
56+
assignst assigns;
57+
get_assigns(local_may_alias, loop_instructions, assigns);
58+
59+
// compute locals
60+
std::unordered_set<irep_idt> loop_locals;
61+
for(const auto &target : loop_instructions)
62+
{
63+
if(target->is_decl())
64+
loop_locals.insert(target->decl_symbol().get_identifier());
65+
}
66+
67+
// widen or drop targets that depend on loop-locals or are non-constant
68+
havoc_utils_is_constantt is_constant(assigns, ns);
69+
assignst result;
70+
for(const auto &expr : assigns)
71+
{
72+
if(depends_on(expr, loop_locals))
73+
{
74+
// Target depends on loop locals, attempt widening to the root object
75+
auto root_object = dfcc_assigns_target_single_root_object(expr, ns, log);
76+
if(!depends_on(root_object, loop_locals))
77+
{
78+
address_of_exprt address_of_root_object(root_object);
79+
address_of_root_object.add_source_location() =
80+
root_object.source_location();
81+
result.emplace(make_object_whole_call_expr(address_of_root_object, ns));
82+
}
83+
}
84+
else
85+
{
86+
address_of_exprt address_of_expr(expr);
87+
address_of_expr.add_source_location() = expr.source_location();
88+
// Target address is not constant, widening to the whole object
89+
if(!is_constant(address_of_expr))
90+
{
91+
result.emplace(make_object_whole_call_expr(address_of_expr, ns));
92+
}
93+
else
94+
{
95+
result.emplace(expr);
96+
}
97+
}
98+
}
99+
100+
return result;
101+
}
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)