-
Notifications
You must be signed in to change notification settings - Fork 39
Consider inlining reduce.c #217
Copy link
Copy link
Closed
Description
Having coefficient reductions in a separate compilation unit comes with an unnecessary performance overhead. Usually link time optimization (LTO) is sufficient to fix that (we do enable it in our benchmarks), but we should not rely on it.
Unless there is a good reason to keep the reductions in a separate CU, we should move them to the header file as inline functions
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels