lattice / quda

QUDA is a library for performing calculations in lattice QCD on GPUs.
https://lattice.github.io/quda
Other
286 stars 94 forks source link

HISQ source files are not listed as dependencies anywhere in the Makefile #69

Closed rbabich closed 12 years ago

rbabich commented 12 years ago

... I don't think (e.g., unitarize_force_quda.cu, unitarize_links_quda.cu, etc.). These should be included in a similar way to the fatlink code.

gshi commented 12 years ago

the hisq/gauge force code (.cu) are included in dslash_quda.cu. The fatlink has an fatlink interface file (.cpp) and that's why it uses the dependency base name in Makefile

rbabich commented 12 years ago

Right, but then they should be listed as dependencies for dslash_quda.o, no?

gshi commented 12 years ago

Good point. They should be listed.

jpfoley commented 12 years ago

I'm confused. Why should they be listed as dependencies? As it stands, a change in any of the hisq .cu files changes dslash_quda.cu.

gshi commented 12 years ago

The Makefile does not know that info. For example, right now if you change hisq_path_force_quda.cu and type "make", it does not recompile dslash_quda.cu. The goal to make it recompile dslash_quda.cu in such case automatically.

jpfoley commented 12 years ago

Oh yeah. I see your point, it's only after the preprocessor runs that dslash_quda.cu is updated. I wasn't thinking straight.