Skip to content
Snippets Groups Projects

Add list_fmap_inj1, Z_to_little_endian_lookup_Some and little_endian_to_Z_spec

Merged Michael Sammler requested to merge msammler/little_endian_spec into master
All threads resolved!

Some more lemmas...

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • added 1 commit

    Compare with previous version

  • I pushed a tweak to the proofs since I couldn't really follow what was going on. For Z_to_little_endian_lookup_Some I now do induction on m (instead of i) as also done in other proofs. For little_endian_to_Z_spec I do induction on the Forall (instead of the list) and perform case distinction on whether i < n. That makes stuff much shorter.

  • I also tried to avoid all: since that's not something we commonly do in std++, and I tried to use rewrite ... by ... when possible.

  • I think the CI will fail on this MR now, but once !342 (merged) is through, and we have a more powerful lia, we should be fine.

  • Thanks for refactoring the proofs!

  • Michael Sammler added 6 commits

    added 6 commits

    Compare with previous version

  • added 1 commit

    • 20fc3620 - list_fmap_inj1 -> list_fmap_inj_1

    Compare with previous version

  • I rebased this branch and now the CI goes through. Should I add a CHANGELOG for this? From my side, I don't think that one is necessary and this MR is ready to go from my side.

    Edited by Michael Sammler
  • Robbert Krebbers resolved all threads

    resolved all threads

  • mentioned in commit 0a8d6074

  • Michael Sammler mentioned in merge request !349 (merged)

    mentioned in merge request !349 (merged)

  • Please register or sign in to reply
    Loading