MergedEdge class for merging node functionality#4603
Conversation
|
Compared to #4600, the impact of this implementation is smaller, but it still has some effects. Specifically, for |
tothtamas28
left a comment
There was a problem hiding this comment.
Please add some unit tests for a KCFG containing MergedEdge instances.
This commit adds these unit tests, but leaves the unit test for minimization for another PR. |
tothtamas28
left a comment
There was a problem hiding this comment.
LGTM. Please test on evm-semantics before merging.
|
cc1e353 to
c39310e
Compare
Please also run integration tests for |
c39310e to
1ce88d7
Compare
|
After spending a long time configuring, I still couldn’t get the evm-semantics integration tests to run. So I decided to submit it as a PR and use the existing CI for testing. |
Here is the alternative implementation for this PR.
MergedEdgeto_rulefunction toMergedEdgefor CSEEdgeforMergedEdgeEdge.to_rulerelated functions for CSE