On 22 Jan 2014, at 00:06, Martin Frb wrote: > So then a patch would be accepted to move the check for > GetNextInstruction(p, hp1) > to the start, and have one single such condition, in one IF block, that will > contain all the others?
The main issue with the peephole optimizer isn't its speed, but its readability, maintainability and extensibility. So unless this significantly improves either (or at the very least doesn't hurt them), I wouldn't modify it. Jonas _______________________________________________ fpc-devel maillist - [email protected] http://lists.freepascal.org/cgi-bin/mailman/listinfo/fpc-devel
