Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bad error localization #68

Open
tdardinier opened this issue Aug 7, 2023 · 0 comments
Open

Bad error localization #68

tdardinier opened this issue Aug 7, 2023 · 0 comments
Labels
pulse Issues related to the Pulse separation logic DSL

Comments

@tdardinier
Copy link
Member

The fold line 252 in the attached file makes verification fail (and commenting it out makes verification succeed). However, the warnings and errors reported are not really helpful. From the command line, I get

...
(Error 19) assertion failed; The SMT solver could not prove the query. Use --query_stats for more details.
../../../../share/steel/examples/pulse/parallel/NewDomains.fst(68,5-68,29): (Warning 249) Losing precision when encoding a function literal: fun a -> NewDomains.unit_emp_stt_pure_pure u#0 a
(Unnannotated abstraction in the compiler ?) (see also ../../../../share/steel/examples/pulse/parallel/NewDomains.fst(65,2-68,27))
../../../../share/steel/examples/pulse/parallel/NewDomains.fst(68,5-251,24): (Warning 249) Losing precision when encoding a function literal: fun v ->
  Pulse.Lib.Reference.pts_to (Mkdtuple4?._3 u#1 u#0 u#0 u#0 (Mkdtuple2?._1 u#1 u#0 t))
    NewDomains.half
    v **
  Pulse.Lib.Core.pure (Some? u#0 v)
...
  • The first problem is the range of warning 249, from line 68 to line 251, while it should only be localized on line 113
  • The second problem is that the actual error 19 (related to the fold on line 252), is reported in VSCode on line 1
@tahina-pro tahina-pro added the pulse Issues related to the Pulse separation logic DSL label Sep 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pulse Issues related to the Pulse separation logic DSL
Projects
None yet
Development

No branches or pull requests

2 participants