Prazek added inline comments. ================ Comment at: include/llvm-c/Core.h:604 @@ -603,3 +603,3 @@ */ -LLVMBool LLVMPrintModuleToFile(LLVMModuleRef M, const char *Filename, +bool LLVMPrintModuleToFile(LLVMModuleRef M, const char *Filename, char **ErrorMessage); ---------------- deadalnix wrote: > Please keep LLVMBool in the C API. ABI change are a NO NO. As I said in my comment, I won't commit most of the changes. This review is only to show what check has found.
http://reviews.llvm.org/D19105 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits