Add support for poison_unsupported_fp64_kernels in API options

The `poison_unsupported_fp64_kernels` option should be represented
as API option. So far the compiler has treated it as Internal option.

Author: matborzyszkowski

IGC/AdaptorOCL/UnifyIROCL.cpp

@@ -438,7 +438,7 @@ static void CommonOCLBasedPasses(
 
         // Check after GlobalDCE in case of doubles in dead functions
         mpm.add(new ErrorCheck());
-        if (pContext->m_InternalOptions.EnableUnsupportedFP64Poisoning) {
+        if (pContext->m_Options.EnableUnsupportedFP64Poisoning) {
             mpm.add(new PoisonFP64Kernels());
         }
 
@@ -563,7 +563,7 @@ static void CommonOCLBasedPasses(
 
     // check for unsupported intrinsics
     mpm.add(new ErrorCheck());
-    if (pContext->m_InternalOptions.EnableUnsupportedFP64Poisoning) {
+    if (pContext->m_Options.EnableUnsupportedFP64Poisoning) {
         mpm.add(new PoisonFP64Kernels());
     }
 

IGC/Compiler/CISACodeGen/OpenCLKernelCodeGen.cpp

@@ -513,13 +513,6 @@ namespace IGC
             }
         }
 
-        if (internalOptions.hasArg(OPT_poison_unsupported_fp64_kernels_common))
-        {
-            // This option forces IGC to poison kernels using fp64
-            // operations on platforms without HW support for fp64.
-            EnableUnsupportedFP64Poisoning = true;
-        }
-
         if (internalOptions.hasArg(OPT_fp64_gen_emu_common))
         {
             // This option enables FP64 emulation for platforms that
@@ -710,6 +703,13 @@ namespace IGC
         {
             Intel256GRFPerThread = true;
         }
+
+        if (apiOptions.hasArg(OPT_poison_unsupported_fp64_kernels_common))
+        {
+            // This option forces IGC to poison kernels using fp64
+            // operations on platforms without HW support for fp64.
+            EnableUnsupportedFP64Poisoning = true;
+        }
     };
 
     unsigned OpenCLProgramContext::GetSlmSizePerSubslice()

IGC/Compiler/CISACodeGen/OpenCLKernelCodeGen.hpp

@@ -93,9 +93,6 @@ namespace IGC
             int FunctionControl                             = -1;
             // Fail comilation if spills are present in compiled kernel
             bool FailOnSpill                                = false;
-            // This option forces IGC to poison kernels using fp64
-            // operations on platforms without HW support for fp64.
-            bool EnableUnsupportedFP64Poisoning             = false;
             // This option enables FP64 emulation for platforms that
             // cannot HW support for double operations
             bool EnableFP64GenEmu                           = false;
@@ -171,6 +168,10 @@ namespace IGC
             // Generic address related
             bool NoLocalToGeneric                   = false;
 
+            // This option forces IGC to poison kernels using fp64
+            // operations on platforms without HW support for fp64.
+            bool EnableUnsupportedFP64Poisoning = false;
+
         private:
             void parseOptions(const char* opts);
         };

IGC/Compiler/Optimizer/OpenCLPasses/ErrorCheckPass.cpp

@@ -111,7 +111,7 @@ void ErrorCheck::visitInstruction(llvm::Instruction& I)
     if (ctx->type == ShaderType::OPENCL_SHADER)
     {
         OpenCLProgramContext *OCLContext = static_cast<OpenCLProgramContext*>(ctx);
-        poisonFP64KernelsEnabled = OCLContext->m_InternalOptions.EnableUnsupportedFP64Poisoning;
+        poisonFP64KernelsEnabled = OCLContext->m_Options.EnableUnsupportedFP64Poisoning;
     }
 
     // chcek that has HW DP support and DP emu is disabled

IGC/Options/include/igc/Options/IGCApiOptions.td

@@ -49,6 +49,9 @@ defm 256_grf_per_thread : CommonFlag<"256-GRF-per-thread">;
 // -cl-intel-greater-than-4GB-buffer-required, -ze-opt-greater-than-4GB-buffer-required
 defm greater_than_4GB_buffer_required : CommonFlag<"greater-than-4GB-buffer-required">;
 
+// -cl-poison-unsupported-fp64-kernels -ze-poison-unsupported-fp64-kernels
+defm poison_unsupported_fp64_kernels : CommonFlag<"poison-unsupported-fp64-kernels">;
+
 // }} Backend API options
 
 // API options from source translation {{

IGC/Options/include/igc/Options/IGCInternalOptions.td

@@ -136,9 +136,6 @@ defm : CommonJoined<"load-cache-default=">, Alias<load_cache_default_common>;
 defm store_cache_default : CommonSeparate<"store-cache-default">;
 defm : CommonJoined<"store-cache-default=">, Alias<store_cache_default_common>;
 
-// -cl-poison-unsupported-fp64-kernels -ze-poison-unsupported-fp64-kernels
-defm poison_unsupported_fp64_kernels : CommonFlag<"poison-unsupported-fp64-kernels">;
-
 // -cl-fp64-gen-emu  -ze-fp64-gen-emu
 defm fp64_gen_emu : CommonFlag<"fp64-gen-emu">;