[Beignet] [PATCH] Turn on OCL_STRICT_CONFORMANCE as default.

Yang Rong rong.r.yang at intel.com
Mon Jun 15 01:35:00 PDT 2015


Also update README.md.

Signed-off-by: Yang Rong <rong.r.yang at intel.com>
---
 backend/src/backend/program.cpp | 2 +-
 docs/Beignet.mdwn               | 6 +++---
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/backend/src/backend/program.cpp b/backend/src/backend/program.cpp
index 5d1ea23..e4cdeaa 100644
--- a/backend/src/backend/program.cpp
+++ b/backend/src/backend/program.cpp
@@ -113,7 +113,7 @@ namespace gbe {
 
 #ifdef GBE_COMPILER_AVAILABLE
   BVAR(OCL_OUTPUT_GEN_IR, false);
-  BVAR(OCL_STRICT_CONFORMANCE, false);
+  BVAR(OCL_STRICT_CONFORMANCE, true);
 
   bool Program::buildFromLLVMFile(const char *fileName, const void* module, std::string &error, int optLevel) {
     ir::Unit *unit = new ir::Unit();
diff --git a/docs/Beignet.mdwn b/docs/Beignet.mdwn
index c0650bb..840f028 100644
--- a/docs/Beignet.mdwn
+++ b/docs/Beignet.mdwn
@@ -195,11 +195,11 @@ Known Issues
 * Precision issue.
   Currently Gen does not provide native support of high precision math functions
   required by OpenCL. We provide a software version to achieve high precision,
-  which you can turn on through
+  which you can turn off through
 
-  `# export OCL_STRICT_CONFORMANCE=1`.
+  `# export OCL_STRICT_CONFORMANCE=0`.
 
-  But be careful, this would make your CL kernel run a little longer.
+  This would lost some precision but gain performance.
 
 * cl\_khr\_gl\_sharing.
   This extension highly depends on mesa support. It seems that mesa would not provide
-- 
1.8.3.2



More information about the Beignet mailing list