lnis-uofu / LSOracle

IDEA project source files
MIT License
97 stars 41 forks source link

Remove duplicate cuts at a node #70

Closed Ravenslofty closed 2 years ago

Ravenslofty commented 2 years ago

While this doesn't improve quality of result in practice, it makes debugging easier and also reduces runtime a bit.

I also took the opportunity to replace assert calls with throws that won't get compiled out in release mode.

Ravenslofty commented 2 years ago

Okay, correction, this improves cavlc and max, but regresses arbiter and sqrt. However, multiplier, sin, priority and voter all map now.

Ravenslofty commented 2 years ago

Most of the above regressions solved by using std::unique from the standard library instead of rolling my own code; cavlc is regressed but the others are improved.