robert-strandh / SICL

A fresh implementation of Common Lisp
Other
1.07k stars 79 forks source link

Allow EQ-AST to appear with one successor. #112

Closed karlosz closed 6 years ago

karlosz commented 6 years ago

This change removes the restriction that EQ-AST must be followed by two successors. Instead, when the case arises that EQ-AST has only one successor, AST-HIR conversion wraps the EQ-AST in an IF. This way, implementers can write CL:EQ as (CLEAVIR-PRIMOP:EQ arg1 arg2) and expect the right HIR graph to be generated, with no redundant branches needed.