Requires in haskell is LHS #2690
Conversation
There was a problem hiding this comment.
The change itself seems fine. I'm not sure the code that was there before was correct, but I guess if it's passing the tests it needs to pass, I could be mistaken. I'm just a little confused about whether it's better to treat the requires clause as "left-hand-side and right-hand side", which is what it is currently doing, or "left-hand-side only", which seems more intuitive.
I'm approving though as it seems to pass the test you wrote.
|
I think the ML encoding is pretty much that requires clause is fully LHS, and ensures is RHS. It's encoded as That's also how the pyk library treats it when building larger transition system structures like CFGs. |
|
And thank you for the fast turnaround @radumereuta |
…untimeverification#2038) * haskell-backend/src/main/native/haskell-backend: 8585c190 - Add class Substitute (runtimeverification#2702) * haskell-backend/src/main/native/haskell-backend: 4f00fdad5 - Profiles rewrite (runtimeverification#2690)
Fixes: #2687