Pass options to compiler during program linking

"-cl-intel-gtpin-rera" and
"-cl-intel-greater-than-4GB-buffer-required"
passed to compiler as internal options

Resolves: NEO-4723

Change-Id: I059027880e9a98aa063f4cd64e84e28311663f46
Signed-off-by: Mateusz Hoppe <mateusz.hoppe@intel.com>

Author: HoppeMateusz

opencl/source/program/link.cpp

@@ -68,6 +68,14 @@ cl_int Program::link(
 
         options = (buildOptions != nullptr) ? buildOptions : "";
 
+        for (const auto &optionString : {CompilerOptions::gtpinRera, CompilerOptions::greaterThan4gbBuffersRequired}) {
+            size_t pos = options.find(optionString);
+            if (pos != std::string::npos) {
+                options.erase(pos, optionString.length());
+                CompilerOptions::concatenateAppend(internalOptions, optionString);
+            }
+        }
+
         if (isKernelDebugEnabled()) {
             appendKernelDebugOptions();
         }

opencl/test/unit_test/program/program_tests.cpp

@@ -1092,6 +1092,34 @@ TEST_P(ProgramFromSourceTest, GivenFlagsWhenCompilingProgramThenBuildOptionsHave
     EXPECT_TRUE(CompilerOptions::contains(cip->buildInternalOptions, pPlatform->getClDevice(0)->peekCompilerExtensions())) << cip->buildInternalOptions;
 }
 
+TEST_F(ProgramTests, GivenFlagsWhenLinkingProgramThenBuildOptionsHaveBeenApplied) {
+    auto cip = new MockCompilerInterfaceCaptureBuildOptions();
+    auto pProgram = std::make_unique<SucceedingGenBinaryProgram>(*pDevice->getExecutionEnvironment());
+    pProgram->setDevice(pDevice);
+    pProgram->sourceCode = "__kernel mock() {}";
+    pProgram->createdFrom = Program::CreatedFrom::SOURCE;
+
+    cl_program program = pProgram.get();
+
+    // compile successfully a kernel to be linked later
+    cl_int retVal = pProgram->compile(0, nullptr, nullptr, 0, nullptr, nullptr, nullptr, nullptr);
+    EXPECT_EQ(CL_SUCCESS, retVal);
+
+    // Ask to link created program with NEO::CompilerOptions::gtpinRera and NEO::CompilerOptions::greaterThan4gbBuffersRequired flags.
+    auto options = CompilerOptions::concatenate(CompilerOptions::greaterThan4gbBuffersRequired, CompilerOptions::gtpinRera, CompilerOptions::finiteMathOnly);
+
+    pDevice->getExecutionEnvironment()->rootDeviceEnvironments[pDevice->getRootDeviceIndex()]->compilerInterface.reset(cip);
+
+    retVal = pProgram->link(0, nullptr, options.c_str(), 1, &program, nullptr, nullptr);
+    EXPECT_EQ(CL_SUCCESS, retVal);
+
+    // Check build options that were applied
+    EXPECT_FALSE(CompilerOptions::contains(cip->buildOptions, CompilerOptions::fastRelaxedMath)) << cip->buildOptions;
+    EXPECT_TRUE(CompilerOptions::contains(cip->buildOptions, CompilerOptions::finiteMathOnly)) << cip->buildOptions;
+    EXPECT_TRUE(CompilerOptions::contains(cip->buildInternalOptions, CompilerOptions::gtpinRera)) << cip->buildInternalOptions;
+    EXPECT_TRUE(CompilerOptions::contains(cip->buildInternalOptions, CompilerOptions::greaterThan4gbBuffersRequired)) << cip->buildInternalOptions;
+}
+
 TEST_P(ProgramFromSourceTest, GivenAdvancedOptionsWhenCreatingProgramThenSuccessIsReturned) {
     std::string testFile;
     size_t sourceSize = 0;

shared/test/unit_test/mocks/mock_compiler_interface.h

@@ -168,6 +168,12 @@ struct MockCompilerInterfaceCaptureBuildOptions : CompilerInterface {
         return this->MockCompilerInterfaceCaptureBuildOptions::compile(device, input, out);
     }
 
+    TranslationOutput::ErrorCode link(const NEO::Device &device,
+                                      const TranslationInput &input,
+                                      TranslationOutput &output) override {
+        return this->MockCompilerInterfaceCaptureBuildOptions::compile(device, input, output);
+    }
+
     std::string buildOptions;
     std::string buildInternalOptions;
 };