mirrors/z3
3
0
Fork 0
mirror of https://github.com/Z3Prover/z3 synced 2026-05-16 07:05:35 +00:00
Code Activity

Fix FPA/BV model-validation soundness bug for Array + Datatype theories (Fixes #9488) (#9500)

Browse source 
* Fix FPA/BV model-validation soundness bug for Array + Datatype theories

* Refactor FPA/BV soundness bug fix to be self-contained in theory_fpa
This commit is contained in:
Adriteyo Das 2026-05-13 05:03:48 +05:30 • committed by GitHub
parent 1a2d3e0ebb
commit fb7a9f83d7
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
1 changed files with 16 additions and 0 deletions
Download patch file Download diff file Expand all files Collapse all files

16
src/smt/theory_fpa.cpp
View file

@ -472,6 +472,22 @@ namespace smt {
wu = m.mk_eq(m_converter.unwrap(wrapped, n->get_sort()), n);
TRACE(t_fpa, tout << "w/u eq: " << std::endl << mk_ismt2_pp(wu, m) << std::endl;);
assert_cnstr(wu);
// For non-FPA-family terms (e.g. datatype accessors like
// get-fp), mk_uf creates a separate BV UF that is not
// linked to bvwrap. Assert wrap(n) == concat(conv_components)
// to close the constraint gap (same pattern as numerals above).
if (n->get_family_id() != get_family_id()) {
expr_ref conv_e = convert(n);
if (m_fpa_util.is_fp(conv_e) && to_app(conv_e)->get_num_args() == 3) {
app_ref conv_a(m);
conv_a = to_app(conv_e.get());
expr_ref cc(m);
cc = m_bv_util.mk_concat({conv_a->get_arg(0), conv_a->get_arg(1), conv_a->get_arg(2)});
assert_cnstr(m.mk_eq(wrapped, cc));
assert_cnstr(mk_side_conditions());
}
}
}
}
}