On 05/11/2013 07:10 PM, Xi Wang wrote: > > This doesn't work for me. > > The problem is that -ldl (`llvm --ldflags`) must come _after_ > -lLLVMSupport (`llvm --libs`). > > Can we move LDFLAGS? > Sorry for the late reply. In that case, your first patch is actually cleaner. I apply your first patch instead. Change pushed. BTW, Pekka, I notice that "sparse-llvm_EXTRA_DEPS" is redundant with the next line, which also have sparse-llvm.o. Am I miss some thing? sparse-llvm_EXTRA_DEPS := sparse-llvm.o sparse-llvm.o $(sparse-llvm_EXTRA_DEPS): BASIC_CFLAGS += $(LLVM_CFLAGS) Chris -- To unsubscribe from this list: send the line "unsubscribe linux-sparse" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html