BuildProgram Optimizations #1 call LinkBuiltInLibrary for the cases i…

…t needs by checking information GLES EXT and replacable builtins pass.

Fix one small issue with builtins enumeration to match the names with OpenCL spec

lib/BuiltinsEnum.h

@@ -225,8 +225,8 @@ enum BuiltinType : unsigned int {
   kLog1p,
   kLogb,
   kMad,
-  kMagmax,
-  kMagmin,
+  kMaxmag,
+  kMinmag,
   kModf,
   kNan,
   kNextafter,

lib/BuiltinsMap.inc

@@ -942,8 +942,8 @@ static std::unordered_map<const char *, Builtins::BuiltinType, cstr_hash,
         {"log1p", Builtins::kLog1p},
         {"logb", Builtins::kLogb},
         {"mad", Builtins::kMad},
-        {"magmax", Builtins::kMagmax},
-        {"magmin", Builtins::kMagmin},
+        {"maxmag", Builtins::kMaxmag},
+        {"minmag", Builtins::kMinmag},
         {"modf", Builtins::kModf},
         {"nan", Builtins::kNan},
         {"nextafter", Builtins::kNextafter},

lib/Compiler.cpp

@@ -67,6 +67,7 @@
 #include <string>
 
 using namespace clang;
+using namespace clspv;
 
 namespace {
 enum class SPIRArch : uint32_t {
@@ -993,8 +994,26 @@ int CompileModule(const llvm::StringRef &input_filename,
     return GenerateIRFile(module, output_buffer);
   }
 
-  if (!LinkBuiltinLibrary(module.get())) {
-    return -1;
+  // check whether any math builtins is available or not
+  // for which we really need to call LinkBuiltinLibrary
+  // e.g. cbrt, hypot etc.
+  bool isBuiltInAvailable = false;
+  for (auto &F : module->getFunctionList()){
+    auto &func_info = clspv::Builtins::Lookup(&F);
+    auto func_type = func_info.getType();
+    if(BUILTIN_IN_GROUP(func_type, Math)){
+      if((Builtins::getExtInstEnum(func_info) == Builtins::kGlslExtInstBad) &&
+      (ReplaceOpenCLBuiltinPass::ReplaceableBuiltins.count(func_type) == 0)){
+        isBuiltInAvailable = true;
+        break;
+      }
+    }
+  }
+
+  // call LinkBuiltinLibrary only if math builtins e.g cbrt, hypot
+  // are available in kernel else ignore it to save few milliseconds
+  if(isBuiltInAvailable && !LinkBuiltinLibrary(module.get())){
+      return -1;
   }
 
   // Run the passes to produce SPIR-V.