Possible typo in Applications' makefile

PASSIONLab / CombBLAS

The Combinatorial BLAS (CombBLAS) is an extensible distributed-memory parallel graph library offering a small but powerful set of linear algebra primitives specifically targeting graph analytics.

Other

59 stars 20 forks source link

Possible typo in Applications' makefile #3

Closed kaansancak closed 3 years ago

kaansancak commented 3 years ago

Should this line be changed as the following $(COMPILER) $(INCADD) $(FLAGS) -c -o mmio.o $(COMBBLAS_SRC)/mmio.c?

I couldn't get it compiled otherwise.

aydinbuluc commented 3 years ago

I don't think so. Which architecture are your trying to compile?

kaansancak commented 3 years ago

I am trying to compile CC.cpp via make cc on Intel(R) Xeon(R) Gold 6226 CPU @ 2.70GHz with gcc/8.3.0 and mvapich2/2.3.2

aydinbuluc commented 3 years ago

This particular makefile is only to be used in NERSC (https://www.nersc.gov/) so it is tailored for that. At NERSC, mmio.c can not be compiled by a C++ compiler (which CC is) and needs a pure C compiler (which cc is). If your modifications to the makefile make the code compilable on your machine, then it is great. But it isn't a typo.

Cheers,

Aydin

kaansancak commented 3 years ago

This particular makefile is only to be used in NERSC (https://www.nersc.gov/) so it is tailored for that. At NERSC, mmio.c can not be compiled by a C++ compiler (which CC is) and needs a pure C compiler (which cc is). If your modifications to the makefile make the code compilable on your machine, then it is great. But it isn't a typo.

Cheers,

Aydin

Thanks for the explanation.

Regards, Kaan