xmos / lib_dsp

Core digital signal processing function library
Other
22 stars 28 forks source link

Double-word alignment of inputs #124

Open larry-xmos opened 6 years ago

larry-xmos commented 6 years ago

The library could do with a way of enforcing double word aligned inputs where double-word instructions such as LDD are used in implementation (eg dsp_filters_biquads)

The problem is that you might write your code one way, say with data on stack, and only then find run into alignment issues, and have to change to another way such as global variables

Could it use a 64bit data type? Or maybe there is a attribute/pragma for it?

henkmuller commented 6 years ago

If we declare dsp_complex_t to have [[aligned(8)]] then that should do it. Always planned to do it - never got around to it.

On 1 Oct 2018, at 17:00, Larry Snizek notifications@github.com wrote:

The library could do with a way of enforcing double word aligned inputs where double-word instructions such as LDD are used in implementation (eg dsp_filters_biquads)

The problem is that you might write your code one way, say with data on stack, and only then find run into alignment issues, and have to change to another way such as global variables

Could it use a 64bit data type? Or maybe there is a attribute/pragma for it?

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub, or mute the thread.

larry-xmos commented 4 years ago

Close old issue – this either hasn't caused enough grief during the almost 2 years or has already been done

xross commented 10 months ago

This is still an issue, see https://www.xcore.com/viewtopic.php?f=47&t=4669&p=40523#p40523