Submitted a patch with the "magic division" and a second patch with a new bench test to showcase the improvements. I hope it is to your satisfaction.
https://bugs.freepascal.org/view.php?id=38806 Gareth aka. Kit On Mon 26/04/21 16:25 , "J. Gareth Moreton" gar...@moreton-family.com sent: > I meant to say "I'm not sure" at the end there. The joys of > writing an e-mail on the phone! > > > Gareth aka. KIt > > > > On Mon 26/04/21 15:14 , "J. Gareth Moreton via fpc-devel" > fpc-devel@lists.freepascal.org sent: > > Indeed. I'm reusing the code that generates > > > > > > the magic constant for me and any additional > > > > > > flags. I've also fixed the bug that caused > > > > > > the failure in my test. Just doing some > > > > > > final cheeks and improving my bench test. > > > > > > > > > > > > It might be possible to make more code cross > > > > > > platform later on. I'mm not due. > > > > > > > > > > > > Gareth aka. Kit > > > > > > > > > > > > On Mon 26/04/21 17:26 , Florian Klämpfl via > > > > > > fpc-develfpc-de...@lists.freepascal.org > > sent: > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Am 26.04.2021 um 08:09 schrieb J. > Gareth > > > > > Moreton > > > > > > > via fpc-devel : > > > > > > > > > > > > > > > > > > > > > > > and the other is implementing > "magic > > > > > > division" > > > > > > > > where division by a constant is > replaced > > > > > with a > > > > > > > multiplication. > > > > > > > > > > > > > > > > > > > > > You are aware there is code for arm 32 bit > > > > > > and e.g. x86 which can most > > > > > > > likely be reused/adapted? It might be even > > > > > > that meanwhile somebody made a > > > > > > > generic implementation of it? > > > > > > > > > > > > > ____________________________________________ > > > > > > ___ > > > > > > > > > > > > > > fpc-devel maillist - fpc- > > > > > > de > > > vel@ > lists.freepascal.org > > > https://lists.freepascal.org/cgi- > > bin/mailman/listinfo/fpc-devel > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > _______________________________________________ > > > > > > fpc-devel maillist - fpc-devel@lists.freepascal.org > > https://lists.freepascal.org/cgi-bin/mailman/listinfo/fpc-devel > > > > > > > > > > > > > > > > > > > > > > _______________________________________________ fpc-devel maillist - fpc-devel@lists.freepascal.org https://lists.freepascal.org/cgi-bin/mailman/listinfo/fpc-devel