-
Notifications
You must be signed in to change notification settings - Fork 199
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Delphi AVX enhancements #88
Comments
I like having IFDEFS... I'll probably be too busy to code AVX support on Delphi for the next 7 months. Open source code can always be forked... With the proper IFDEFs, it could work. |
If you are interested in that I can provide you Delphi units and contribute a bit to your great work. Actually I already created one that computes the dot product (as far as I can see the working horse...) for Delphi 32 and 64 bit (+Unix ABI). And... these units include A LOT of Ifdefs ;) :D |
joaopauloschuler#85: Fixed opencl problem + Better cloning (way faster) + Fixed many Delphi warnings + New convolutional classes with swift and gelu + Automatic check if AVX is available + Fixed issues with output procedures: sometimes error proc pointed to the console output (althouth I wanted that in a file...)
I actually managed to fork the repository and add the Delphi AVX enhancements... though I wasn't able to create a pull request... anyway if you are interested in the changes check out: |
@mikerabat , I'm certainly interested. I'll try to figure why the pull request didn't work. |
@mikerabat , I can see plenty of coding. It will take me some time to test it in all supported environments. Most likely, I'll ask you to create a new branch with smaller commits that are easier to merge. We'll see. I'll FUP in few months from now. |
sure thing :) I'll try my best to help out. |
This request would be twofold...
First it would be great if there are AVX functions for the Delphi community too. I actually started to implement the DotProduct for
Delphi2010 and upwards but I have no clue how to get that back to you.
The second suggestion is to get rid of the AVX defines and let the class decide which path would be the fastest to use.
E.g. the unit I wrote once on https://github.com/mikerabat/mrmath/blob/master/CPUFeatures.pas is perfectly suited for that ;)
Let me know what you think of this idea.
The text was updated successfully, but these errors were encountered: