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

Post-increment floating-point loads and stores #386

Open
andreaskurth opened this issue Jun 26, 2020 · 1 comment
Open

Post-increment floating-point loads and stores #386

andreaskurth opened this issue Jun 26, 2020 · 1 comment
Labels
Component:RTL For issues in the RTL (e.g. for files in the rtl directory) PARAM:FPU Issue depends on the FPU parameter Type:Enhancement For feature requests and enhancements WAIVED:CV32E40P Issue does not impact a major release of CV32E40P and is waived

Comments

@andreaskurth
Copy link
Contributor

andreaskurth commented Jun 26, 2020

Is it correct that the post-increment extension currently only supports integer loads and stores but not floating-point loads and stores? If so, are there any plans for enabling that extension for floating-point loads and stores? That would help increase the utilization of the FPU (depending on the hot loop even very significantly) and therefore floating-point performance.

@Silabs-ArjanB
Copy link
Contributor

Hi @accuminium, your understanding is correct. There are no plans for CV32E40P to add a similar extension for floating-point loads and stores. (I would assume though (but did not try it out) that you get what you want when you would use the PULP_ZFINX = 1 parameter together with integer load/stores).

@Silabs-ArjanB Silabs-ArjanB added Component:RTL For issues in the RTL (e.g. for files in the rtl directory) PARAM:FPU Issue depends on the FPU parameter Type:Enhancement For feature requests and enhancements WAIVED:CV32E40P Issue does not impact a major release of CV32E40P and is waived labels Jul 16, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component:RTL For issues in the RTL (e.g. for files in the rtl directory) PARAM:FPU Issue depends on the FPU parameter Type:Enhancement For feature requests and enhancements WAIVED:CV32E40P Issue does not impact a major release of CV32E40P and is waived
Projects
None yet
Development

No branches or pull requests

2 participants