commit a624c0c8c9540d3adf4ad5eeff1efbaa44ca77df
author Kévin Petit <kpet@free.fr> Tue May 07 20:27:43 2019 +0800
committer alan-baker <33432579+alan-baker@users.noreply.github.com> Tue May 07 08:27:43 2019 -0400
tree 97a97976ac6afa2293961e579abbb429f05a8539
parent 225164780e85b4cd9ad5d781d20ff8531f49f837

Add an option to undefine __IMAGE_SUPPORT__ (#353)

Clang now defines it by default (was picked up in the update)
and the conformance tests check that this is defined if and only
if the runtime advertises support.

Signed-off-by: Kévin Petit <kpet@free.fr>

lib/Compiler.cpp

diff --git a/lib/Compiler.cpp b/lib/Compiler.cpp
index e94eb59..c222a1e 100644
--- a/lib/Compiler.cpp
+++ b/lib/Compiler.cpp
@@ -429,7 +429,9 @@
   instance.getLangOpts().FastRelaxedMath = cl_fast_relaxed_math;
 
   // Preprocessor options
-  instance.getPreprocessorOpts().addMacroDef("__IMAGE_SUPPORT__");
+  if (!clspv::Option::ImageSupport()) {
+    instance.getPreprocessorOpts().addMacroUndef("__IMAGE_SUPPORT__");
+  }
   if (cl_fast_relaxed_math) {
     instance.getPreprocessorOpts().addMacroDef("__FAST_RELAXED_MATH__");
   }