Remove CL_CONTEXT_COMPILER_MODE_SIMULATION_INTELFPGA

This compiler mode (simulation mode, mode 6) is unused by the runtime and hence safe to remove. I am removing it so that the differences between simulation devices and hardware devices are removed as much as possible.

When we search for the compiler mode on Google, nothing returned, so it's also not documented anywhere.

Tests Done:
Runtime Unit Tests
Checked that products/regtest/hld does not have lines like "compiler_mode => 6" or "compiler_mode => '6'"

Author: intel-liudean

include/acl_types.h

@@ -198,8 +198,7 @@ typedef cl_ulong acl_object_magic_t;
 //  scenario during runtime.
 #define CL_CONTEXT_COMPILER_MODE_ONLINE_INTELFPGA 4
 #define CL_CONTEXT_COMPILER_MODE_SPLIT_KERNEL_INTELFPGA 5
-#define CL_CONTEXT_COMPILER_MODE_SIMULATION_INTELFPGA 6
-#define CL_CONTEXT_COMPILER_MODE_NUM_MODES_INTELFPGA 7
+#define CL_CONTEXT_COMPILER_MODE_NUM_MODES_INTELFPGA 6
 
 typedef enum {
   ACL_COMPILER_MODE_OFFLINE = CL_CONTEXT_COMPILER_MODE_OFFLINE_INTELFPGA,
@@ -212,7 +211,6 @@ typedef enum {
   ACL_COMPILER_MODE_ONLINE = CL_CONTEXT_COMPILER_MODE_ONLINE_INTELFPGA,
   ACL_COMPILER_MODE_SPLIT_KERNEL =
       CL_CONTEXT_COMPILER_MODE_SPLIT_KERNEL_INTELFPGA,
-  ACL_COMPILER_MODE_SIMULATION = CL_CONTEXT_COMPILER_MODE_SIMULATION_INTELFPGA,
   ACL_COMPILER_MODE_NUM_MODES = CL_CONTEXT_COMPILER_MODE_NUM_MODES_INTELFPGA
 } acl_compiler_mode_t;
 

src/acl_program.cpp

@@ -1349,7 +1349,7 @@ static cl_int l_build_program_for_device(cl_program program,
     }
     // num_global_mem_systems of device binary autodiscovery def will remain 0
     // only when CL_CONTEXT_COMPILER_MODE_INTELFPGA is
-    // ACL_COMPILER_MODE_PRELOADED_BINARY_ONLY or ACL_COMPILER_MODE_SIMULATION,
+    // ACL_COMPILER_MODE_PRELOADED_BINARY_ONLY,
     // in these situations we should only use num_global_memory_system of device
     // autodiscovery def which would be initialized with acl_kernel_if_init by
     // reading the autodiscovery string of the bitstream currently programmed on

test/acl_context_test.cpp

@@ -948,7 +948,7 @@ MT_TEST(Context, compiler_mode) {
       {"x", false, false}, {"x1", false, false}, {"-1", false, false},
       {"0", true, true},   {"1", true, true},    {"2", true, true},
       {"3", true, true},   {"4", true, true},    {"5", true, true},
-      {"6", true, true},   {"7", true, false}};
+      {"6", true, false}};
 
   for (unsigned ienv = 0; ienv < sizeof(envvals) / sizeof(envvals[0]); ienv++) {
     // don't set env vars until all threads are ready to use it