search

npadmana / DistributedFFT

6 stars 2 forks source link

Current code destroys source array #26

Closed npadmana closed 5 years ago

npadmana commented 5 years ago

The current version of the code overwrites the source array. This isn't ideal (it doesn't hurt the NPB benchmark because of the way it is structured), but could quickly become expensive since it requires a temporary copy in addition to the work array.

It's probably worth a ~small~ hit to performance to fix this.

npadmana commented 5 years ago

Branch elegant should fix this. Still need to test....

npadmana commented 5 years ago

This is now explicitly tested in the elegance-cleanup-2 branch by target/Dist/test_simple (and works).