Skip to content

Simplify fastmove.inc: rely on “inline” and don’t handle FPC_PIC explicitly.

Rika requested to merge runewalsh/source:fm into main

The way fastmove.inc is implemented in the part of calling fastmoveproc from assembly is the remnant of the old fastmove.inc which probably only did it because at that time, Delphi did not support inline. It leads to complications that are removed by this MR, and I guess I can finally remove the if not defined(darwin) condition on including it (can I???).

It makes things slightly better (1 less jump taken), IF (WHEN) Move call is inlined; otherwise things become worse (1 jump changed to 1 call; FPC could do it itself, in which case results without inlining would be the same as before the MR #missing_optimization). Usually, inlining procs, since I asked nicely.

Merge request reports