New GC interface. Added changes from code review.

Author: AndreyPavlenko

CMakeLists.txt

@@ -102,6 +102,6 @@ add_mlir_library(graph_compiler SHARED ${GC_LIB_SOURCES})
 target_include_directories(graph_compiler PUBLIC ${GC_LIB_INCLUDES})
 target_compile_options(graph_compiler PRIVATE -fvisibility=hidden -fexceptions)
 target_link_options(graph_compiler PRIVATE -Wl,--gc-sections)
-target_link_libraries(graph_compiler PUBLIC ${GC_LIB_LINKED_LIBS})
+target_link_libraries(graph_compiler PRIVATE ${GC_LIB_LINKED_LIBS})
 
 add_subdirectory(test)

src/dnnl/dnnl_graph_compiler.cpp

@@ -210,13 +210,11 @@ GC_DLL_EXPORT dnnl_status_t dnnl_graph_compiler_compile(
 }
 
 GC_DLL_EXPORT void dnnl_graph_compiler_destroy_executable(
-    const struct dnnl_graph_compiler *gc,
     const struct dnnl_graph_compiler_executable *exe) {
   delete exe;
 }
 
 GC_DLL_EXPORT dnnl_status_t dnnl_graph_compiler_execute(
-    const struct dnnl_graph_compiler *gc,
     const struct dnnl_graph_compiler_executable *exe,
     dnnl_graph_compiler_tensor *inputs, dnnl_graph_compiler_tensor *outputs) {
   try {

test/dnnl/DnnlTestUtils.h

@@ -1,10 +1,20 @@
-//===-- DnnlTestUtils.h - Test utils ----------------------------*- C++ -*-===//
-//
-// This file is licensed under the Apache License v2.0 with LLVM Exceptions.
-// See https://llvm.org/LICENSE.txt for license information.
-// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
-//
-//===----------------------------------------------------------------------===//
+/*
+ * Copyright (C) 2024 Intel Corporation
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions
+ * and limitations under the License.
+ *
+ * SPDX-License-Identifier: Apache-2.0
+ */
 
 #include <filesystem>
 #include <fstream>

test/dnnl/TestApiBasic.cpp

@@ -1,10 +1,20 @@
-//===-- TestApiBasic.cpp - Basic tests for the dnnl interface ---*- C++ -*-===//
-//
-// This file is licensed under the Apache License v2.0 with LLVM Exceptions.
-// See https://llvm.org/LICENSE.txt for license information.
-// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
-//
-//===----------------------------------------------------------------------===//
+/*
+ * Copyright (C) 2024 Intel Corporation
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions
+ * and limitations under the License.
+ *
+ * SPDX-License-Identifier: Apache-2.0
+ */
 
 #include <gtest/gtest.h>
 
@@ -37,10 +47,9 @@ TEST(TestApiBasic, basicWorkflow) {
     arg2[i] = i;
   }
 
-  ASSERT_EQ(dnnl_graph_compiler_execute(gc, exe, inputs, outputs),
-            dnnl_success);
+  ASSERT_EQ(dnnl_graph_compiler_execute(exe, inputs, outputs), dnnl_success);
 
-  dnnl_graph_compiler_destroy_executable(gc, exe);
+  dnnl_graph_compiler_destroy_executable(exe);
   dnnl_graph_compiler_destroy(gc);
 
   for (size_t i = 0; i < 128; i++) {

test/dnnl/TestJsonParser.cpp

@@ -1,10 +1,20 @@
-//===-- TestJsonParser.cpp - JsonParser test --------------------*- C++ -*-===//
-//
-// This file is licensed under the Apache License v2.0 with LLVM Exceptions.
-// See https://llvm.org/LICENSE.txt for license information.
-// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
-//
-//===----------------------------------------------------------------------===//
+/*
+ * Copyright (C) 2024 Intel Corporation
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions
+ * and limitations under the License.
+ *
+ * SPDX-License-Identifier: Apache-2.0
+ */
 
 #include <gtest/gtest.h>