[Beignet] [PATCH 3/3] fix build error log not output issue.

Pan, Xiuli xiuli.pan at intel.com
Wed Mar 8 06:23:48 UTC 2017


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-bounces at lists.freedesktop.org] On Behalf Of xionghu.luo at intel.com
Sent: Friday, March 3, 2017 12:08 AM
To: beignet at lists.freedesktop.org
Cc: Luo, Xionghu <xionghu.luo at intel.com>
Subject: [Beignet] [PATCH 3/3] fix build error log not output issue.

From: Luo Xionghu <xionghu.luo at intel.com>

Signed-off-by: Luo Xionghu <xionghu.luo at 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 at lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/beignet


More information about the Beignet mailing list