Skip to content
Snippets Groups Projects

Add instance for `FromSep (tc_opaque P) Q1 Q2`

Merged Robbert Krebbers requested to merge robbert/from_sep_opaque_fix into master

See discussion at !833 (diffs, comment 83516)

  • Turns out the cost/comment for FromAnd is obsolete, FromAnd is never used by iCombine
  • I removed the Hint Mode for FromAnd that corresponds to it being used by iCombine
  • Turns out there was an instance for missing for tc_opaque + FromSep
  • I added that instance with the correct cost + a better comment + a test case that used to fail prior to this MR

Merge request reports

Merge request pipeline #70456 passed

Merge request pipeline passed for f7255e63

Approval is optional

Merged by Ralf JungRalf Jung 2 years ago (Aug 9, 2022 6:18pm UTC)

Merge details

  • Changes merged into master with 7827fc3f.
  • Deleted the source branch.

Pipeline #70468 passed

Pipeline passed for 7827fc3f on master

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading