I think llvm::errs() << options; should be llvm::errs() << err; And there seems some more this kind of bugs in program.cpp. Such as programCompileFromSource, no need for output options. For there is no reason to output some options here.
-----Original Message----- From: Beignet [mailto:beignet-boun...@lists.freedesktop.org] On Behalf Of xionghu....@intel.com Sent: Friday, March 3, 2017 12:08 AM To: beignet@lists.freedesktop.org Cc: Luo, Xionghu <xionghu....@intel.com> Subject: [Beignet] [PATCH 3/3] fix build error log not output issue. From: Luo Xionghu <xionghu....@intel.com> Signed-off-by: Luo Xionghu <xionghu....@intel.com> --- backend/src/backend/program.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/backend/src/backend/program.cpp b/backend/src/backend/program.cpp index 09c79d8..5f18dd3 100644 --- a/backend/src/backend/program.cpp +++ b/backend/src/backend/program.cpp @@ -1114,7 +1114,7 @@ EXTEND_QUOTE: stringSize, err, errSize, optLevel, options); if (err != NULL) *errSize += clangErrSize; - if (OCL_OUTPUT_BUILD_LOG && options) + if (OCL_OUTPUT_BUILD_LOG && err) llvm::errs() << options; } else p = NULL; -- 2.5.0 _______________________________________________ Beignet mailing list Beignet@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/beignet _______________________________________________ Beignet mailing list Beignet@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/beignet
