🚀 go-pugleaf

david r chase <chase@world.std.com> wrote:
> D. J. Bernstein <djb@silverton.berkeley.edu> wrote:
> > Joe Buck <jbuck@synopsys.com> wrote:
> > > 	/* now res is at most 32767+4 */
> > > 	return (res < 32767) ? res : res - 32767;
> > Branching is bad. Look at the routines that use res, and rewrite them so
> > that they won't mind inputs up to 32771.
> Branches aren't necesssary, nor are they necessarily bad.

For some machines, yes. For others this comparison will nearly triple
the run time of the subroutine.

But you missed my main point, which is that the code should be deleted
entirely. Your five instructions should be ZERO instructions.

---Dan