Re: [Starpu-devel] [PATCH] examples audio fix

[Francisco Alecrim <alecrim@gmail.com>]

Hi

Em qui, 15 de jun de 2017 às 18:19, Samuel Thibault <samuel.thibault@inria.fr> escreveu:

Hello,

Francisco Alecrim, on jeu. 15 juin 2017 14:44:39 -0300, wrote:
> define nite as global variable because old scope was not reacheable by
> starpu_data_filter,

Mmm, I don't see how the old scope would not be reachable by
starpu_data_filter.

You're right! This change does not make sense.

I got the error below and ctags was pushing me to niter definition at "gcc-plugin/examples/stencil5.c". So I thought it was related with scope.  I reverted the code and now I'm with the error again. Working on it. Thanks!
[starpu][_starpu_data_partition][assert failure] Partitioning data 0x5647bd05b9d0 in 0 piece does not make sense
starpu_audio_processing: ../../src/datawizard/filters.c:170: _starpu_data_partition: Assertion `nparts > 0' failed.
Aborted (core dumped)
 

> cublas fix

Indeed, thanks!

> and use CFLAGS variable on Makefile.

I didn't see that part in the patch?

LDFLAGS was empty for me.

diff --git a/examples/audio/Makefile b/examples/audio/Makefile
index 8019697..37ce6bf 100644
--- a/examples/audio/Makefile
+++ b/examples/audio/Makefile
@@ -27,7 +27,7 @@ all: starpu_audio_processing
 starpu_audio_processing.o: starpu_audio_processing.c
 
 starpu_audio_processing: starpu_audio_processing.o
-       $(CC) $(LDFLAGS) starpu_audio_processing.o -o starpu_audio_processing $(LIBS)
+       $(CC) $(CFLAGS) starpu_audio_processing.o -o starpu_audio_processing $(LIBS)
 
 clean:
        rm -f *.o

 

Samuel

--

Francisco Keppler Silva Alecrim

alecrim@gmail.com