ANL-CESAR/XSBench

Compilation error

Closed this issue · 3 comments

mquinson commented

Hello,

I am one of the maintainer of the SMPI simulator, a tool allowing to run real MPI application on top of simulated infrastructure. More info here, in particular on slides 45 and 46.

We run nightly tests against most of the applications listed as a ProxyApp by the Exascale Project, and as you certainly know, XSBench is one of them. For more info, head to our compilation farm.

Since approximately 2 days, your code fails to compile with the following error:

smpicc -std=gnu99 -Wall -fopenmp -flto -O3 -DMPI -c CalculateXS.c -o CalculateXS.o
/home/ci/workspace/SMPI-proxy-apps/build_mode/SMPI/label/proxy-apps/Benchmarks/ECP/XSBench/src/Main.c: In function ‘main’:
/home/ci/workspace/SMPI-proxy-apps/build_mode/SMPI/label/proxy-apps/Benchmarks/ECP/XSBench/src/Main.c:40:6: error: ‘thread’ undeclared (first use in this function)
  if( thread == 0 )
      ^~~~~~
/home/ci/workspace/SMPI-proxy-apps/build_mode/SMPI/label/proxy-apps/Benchmarks/ECP/XSBench/src/Main.c:40:6: note: each undeclared identifier is reported only once for each function it appears in
/home/ci/workspace/SMPI-proxy-apps/build_mode/SMPI/label/proxy-apps/Benchmarks/ECP/XSBench/src/Main.c:71:8: warning: unused variable ‘index_data’ [-Wunused-variable]
  int * index_data = NULL;
        ^~~~~~~~~~
/home/ci/workspace/SMPI-proxy-apps/build_mode/SMPI/label/proxy-apps/Benchmarks/ECP/XSBench/src/Main.c:19:13: warning: unused variable ‘stat’ [-Wunused-variable]
  MPI_Status stat;
             ^~~~
Makefile:129: recipe for target 'Main.o' failed

I suspect that this is related to the changes that you introduced recently, isn't it?

Thanks a lot for maintaining this proxy app, that's really precious for the tool makers.
Martin.

mquinson commented

Sorry, my mistake, the bug seem to be related to our patch, that still applies to the recent version, but does not compile anymore. Sorry.

jtramm commented

Hey Martin,

Glad you were able to get this one fixed. Let me know if you run into any more issues with the new version!

-John

mquinson commented