google-research / dex-lang

Research language for array processing in the Haskell/ML family
BSD 3-Clause "New" or "Revised" License
1.58k stars 107 forks source link

Make a hard distinction between CoreIR and SimpIR, and disallow Lam/Pi in the latter #1179

Closed dougalm closed 1 year ago

dougalm commented 1 year ago

Could you / we also fix the comments at the top of Simplify.hs? I think a reasonable end state would be to mention only additional invariants not captured by the IR parameters, and fix references to PrimHof (since it no longer exists).

My follow-up change will capture ~all the invariants in the IR parameters, at which point we can delete the entire comment. In the meantime. Let's not worry about carefully updating the comment to reflect the temporary intermediate state before then.