egraphs-good / eggcc

MIT License
42 stars 8 forks source link

Add Expr-size #286

Closed clyben closed 8 months ago

clyben commented 8 months ago

This PR add Expr-size rules that compute the size of a Expr, which Loop invariant hoisting depends on. Something I'm concerning is what should be counted. I don't count the Cons and Nil constructor in List, and also don't count the Get and All constructor. Alternative interpretations could be:

clyben commented 8 months ago

@oflatt Do you think any alternative Expr-size mentioned above would make more sense, or this one is (not counting list, Get, and All) is good for now?