Skip to content
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

fp::fma should use fmuladd #112

Closed
gnzlbg opened this issue Aug 31, 2018 · 0 comments
Closed

fp::fma should use fmuladd #112

gnzlbg opened this issue Aug 31, 2018 · 0 comments
Labels
Enhancement New feature or request P-Medium Performance Something isn't fast

Comments

@gnzlbg
Copy link
Contributor

gnzlbg commented Aug 31, 2018

Currently the floating-point fma vector method performs a fused-multiply add unconditionally, even in targets for which it is slower, or in targets where it is not supported where it has to be emulated (no idea how reliable this emulation is).

It might be better to do a conditional fused-multiply add (llvm.fmuladd) here instead. Those who want to perform a fused-multiply add with extended precision and have its guarantees should probably just use std::arch.

@gnzlbg gnzlbg added Enhancement New feature or request Performance Something isn't fast P-Medium labels Aug 31, 2018
@gnzlbg gnzlbg changed the title .fma should use fmuladd instead of fma .fma should use fmuladd Aug 31, 2018
@gnzlbg gnzlbg changed the title .fma should use fmuladd fp::fma should use fmuladd Aug 31, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement New feature or request P-Medium Performance Something isn't fast
Projects
None yet
Development

No branches or pull requests

1 participant