commit 508dbfbadf01eb8f34e86fc5bf44afb3d51f6ffe
author Ben Vanik <benvanik@google.com> Mon Jun 21 09:20:41 2021 -0700
committer Ben Vanik <benvanik@google.com> Mon Jun 21 09:49:56 2021 -0700
tree 4ccc403dd92efc0eb43e01cdddca78b101923b68
parent 98aaac35418bd9cd7fc3388c85c71973bd0799f8

Fixing check module factory fn name.

iree/modules/check/check_test.cc

diff --git a/iree/modules/check/check_test.cc b/iree/modules/check/check_test.cc
index 30c5e9c..daab092 100644
--- a/iree/modules/check/check_test.cc
+++ b/iree/modules/check/check_test.cc
@@ -49,7 +49,7 @@
         iree_vm_instance_create(iree_allocator_system(), &instance_));
 
     IREE_ASSERT_OK(
-        check_native_module_create(iree_allocator_system(), &check_module_))
+        iree_check_module_create(iree_allocator_system(), &check_module_))
         << "Native module failed to init";
   }
 

iree/modules/check/module.cc

diff --git a/iree/modules/check/module.cc b/iree/modules/check/module.cc
index 2a14530..19348eb 100644
--- a/iree/modules/check/module.cc
+++ b/iree/modules/check/module.cc
@@ -4,6 +4,8 @@
 // See https://llvm.org/LICENSE.txt for license information.
 // SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
 
+#include "iree/modules/check/module.h"
+
 #include <cassert>
 #include <cmath>
 #include <cstdint>
@@ -19,7 +21,6 @@
 #include "iree/base/internal/math.h"
 #include "iree/base/status.h"
 #include "iree/hal/api.h"
-#include "iree/modules/check/module.h"
 #include "iree/modules/hal/module.h"
 #include "iree/testing/gtest.h"
 #include "iree/vm/native_module_cc.h"
@@ -407,7 +408,7 @@
 
 // Note that while we are using C++ bindings internally we still expose the
 // module as a C instance. This hides the details of our implementation.
-extern "C" iree_status_t check_native_module_create(
+extern "C" iree_status_t iree_check_module_create(
     iree_allocator_t allocator, iree_vm_module_t** out_module) {
   IREE_ASSERT_ARGUMENT(out_module);
   *out_module = NULL;

iree/modules/check/module.h

diff --git a/iree/modules/check/module.h b/iree/modules/check/module.h
index 5264b49..24d29ba 100644
--- a/iree/modules/check/module.h
+++ b/iree/modules/check/module.h
@@ -17,8 +17,8 @@
 #endif  // __cplusplus
 
 // Creates a native custom module.
-iree_status_t check_native_module_create(iree_allocator_t allocator,
-                                         iree_vm_module_t** out_module);
+iree_status_t iree_check_module_create(iree_allocator_t allocator,
+                                       iree_vm_module_t** out_module);
 
 #ifdef __cplusplus
 }  // extern "C"

iree/tools/iree-check-module-main.cc

diff --git a/iree/tools/iree-check-module-main.cc b/iree/tools/iree-check-module-main.cc
index b2a65b5..9d58f56 100644
--- a/iree/tools/iree-check-module-main.cc
+++ b/iree/tools/iree-check-module-main.cc
@@ -110,7 +110,8 @@
   IREE_RETURN_IF_ERROR(
       iree_hal_module_create(device, iree_allocator_system(), &hal_module));
   iree_vm_module_t* check_module = nullptr;
-  check_native_module_create(iree_allocator_system(), &check_module);
+  IREE_RETURN_IF_ERROR(
+      iree_check_module_create(iree_allocator_system(), &check_module));
 
   std::array<iree_vm_module_t*, 3> modules = {hal_module, check_module,
                                               input_module};