search

robeverest / cufft

BSD 3-Clause "New" or "Revised" License
8 stars 5 forks source link

configure: separate LDFLAGS and extra-libraries fields #6

Closed tmcdonell closed 10 years ago

tmcdonell commented 10 years ago

This correctly addresses robeverest/cufft#4

The problem with the combined LDFLAGS macro is that this is expanded too early on the command line. Namely, we have -lcufft placed before the object files that use these symbols. Since the linker has not yet encountered any uses of cufft symbols at that time, in incorrectly assumes that it is not used and strips all the symbols.

See also tmcdonell/cuda#13

tmcdonell commented 10 years ago

The commit is a little noisy, sorry. I (mostly) just copied the configure.ac from the cuda package.

robeverest commented 10 years ago

Ah, thanks. That explains all the linking problems we've had.