Skip to content
Snippets Groups Projects

Pinning

Open Michael Sammler requested to merge ci/pinning into master

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

    • fc9a61b7 - Add pinning to model of types

    Compare with previous version

  • added 1 commit

    • a7aba740 - Add pinning to model of types

    Compare with previous version

  • added 1 commit

    • 879a2f95 - Add pinning to model of types

    Compare with previous version

  • Ralf Jung
  • Michael Sammler mentioned in merge request stdpp!235 (merged)

    mentioned in merge request stdpp!235 (merged)

  • Michael Sammler added 4 commits

    added 4 commits

    Compare with previous version

  • added 1 commit

    • e9e3f2ed - Add pinning to model of types

    Compare with previous version

  • added 1 commit

    • 9791328f - Add pinning to model of types

    Compare with previous version

  • Michael Sammler resolved all threads

    resolved all threads

  • Michael Sammler marked this merge request as ready

    marked this merge request as ready

  • Michael Sammler added 2 commits

    added 2 commits

    • 9f72ef66 - 1 commit from branch master
    • 2e7d222c - Add pinning to model of types

    Compare with previous version

  • Ralf Jung
  • Ralf Jung
    Ralf Jung @jung started a thread on the diff
  • 18 18 Definition shrN := lrustN .@ "shr".
    19 19
    20 20 Definition thread_id := na_inv_pool_name.
    21 Inductive pin_state := | Pinned | NotPinned.
    22
    23 Lemma match_pin_state {A} ps (P : A):
    24 match ps with | Pinned | _ => P end = P.
    25 Proof. by destruct ps. Qed.
    • This is a strange lemma, and the name isn't very clear either... could you make it Local at least to clarify that this is not something one has to care about much?

    • I've changed the name and made it local. Please have a look if it is better now.

    • Hm, I can't really make sense of the new name either. Nor can I think of anything better... would something involving eta make sense?

    • Please register or sign in to reply
  • Ralf Jung
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading