ComputationalRadiationPhysics / graybat

Graph Approach for Highly Generic Communication Schemes Based on Adaptive Topologies :satellite:
Other
8 stars 4 forks source link

Move from g++ to clang++ #9

Closed erikzenker closed 9 years ago

erikzenker commented 9 years ago

Because of the nice error output :feelsgood:

ax3l commented 9 years ago

travis supports testing both - so why not add testing both with drone.io :)

erikzenker commented 9 years ago

Good point. So there have to be some modifications in the cmake file.

ax3l commented 9 years ago

oh I see what you mean. holy moly, never write such a line :) the user and environment controls the compiler, not the cmake/configure script ;)

export CXX=`which g++`

or stuff like

CXX=`which clang` cmake ~/src/grayBat
ax3l commented 9 years ago

and what does DO not compile with -OX with x > 1, can lead to segfaults mean ? ;) valgrind it! :)

erikzenker commented 9 years ago

There was some stupid MPI failure made from some stupid programmer of this project. Thus, -03 is not a problem anymore. You are right about this CXX issue, I hard coded that line because of some missconfiguration of some cluster in the forest (do not want to blame anyone here :space_invader:)

ax3l commented 9 years ago

hehe, all right.

well, you can even force the forest-cluster to work with environment vars ;)