Add instance for `FromSep (tc_opaque P) Q1 Q2`
See discussion at !833 (diffs, comment 83516)
- Turns out the cost/comment for
FromAnd
is obsolete,FromAnd
is never used byiCombine
- I removed the
Hint Mode
forFromAnd
that corresponds to it being used byiCombine
- 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
Activity
Filter activity
assigned to @robbertkrebbers
mentioned in merge request !833 (merged)
added 20 commits
-
df473eee...55e78d6f - 17 commits from branch
master
- 4179bb8a - `FromAnd` is never used for `iCombine`, so remove that mode.
- f8667d35 - Add instance for `FromSep (tc_opaque P) Q1 Q2` and update comment.
- f7255e63 - Add test.
Toggle commit list-
df473eee...55e78d6f - 17 commits from branch
mentioned in commit 7827fc3f
Please register or sign in to reply