Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Introduce centralised name transformation functions #9088

Merged
merged 3 commits into from
Sep 30, 2021
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
98 changes: 98 additions & 0 deletions python/tvm/relay/backend/name_transforms.py
Original file line number Diff line number Diff line change
@@ -0,0 +1,98 @@
# Licensed to the Apache Software Foundation (ASF) under one
# or more contributor license agreements. See the NOTICE file
# distributed with this work for additional information
# regarding copyright ownership. The ASF licenses this file
# to you 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.
"""
Name transformation functions for use in code generation
"""

from typing import List, Union

from tvm import TVMError
from . import _backend


def to_c_function_style(original_name: str):
"""Transform a name to the C function style assuming it is
appropriately constructed using the prefixing functions

Parameters
----------
original_name : str
Original name to transform
"""
return _backend.ToCFunctionStyle(original_name)


def to_c_variable_style(original_name: str):
"""Transform a name to the C variable style assuming it is
appropriately constructed using the prefixing functions

Parameters
----------
original_name : str
Original name to transform
"""
return _backend.ToCVariableStyle(original_name)


def _preprocess_names(names: Union[List[str], str]):
"""Preprocesses name strings into format for C++ functions

Parameters
----------
names : Union[List[str], str]
List of names to combine to form a combined name or the name itself
"""
if isinstance(names, str):
if names == "":
raise TVMError("Name is empty")
return [names]
return names


def prefix_name(names: Union[List[str], str]):
"""Apply TVM-specific prefix to a function name

Parameters
----------
names : Union[List[str], str]
List of names to combine to form a combined name or the name itself
"""

return _backend.PrefixName(_preprocess_names(names))


def prefix_generated_name(names: Union[List[str], str]):
"""Apply generated TVM-specific prefix to a function name

Parameters
----------
names : Union[List[str], str]
List of names to combine to form a combined name or the name itself
"""

return _backend.PrefixGeneratedName(_preprocess_names(names))


def sanitize_name(original_name: str):
"""Sanitize name for output into compiler artifacts

Parameters
----------
original_name : str
Original name to sanitize
"""
return _backend.SanitizeName(original_name)
104 changes: 104 additions & 0 deletions src/relay/backend/name_transforms.cc
Original file line number Diff line number Diff line change
@@ -0,0 +1,104 @@
/*
* Licensed to the Apache Software Foundation (ASF) under one
* or more contributor license agreements. See the NOTICE file
* distributed with this work for additional information
* regarding copyright ownership. The ASF licenses this file
* to you 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.
*/

#include "name_transforms.h"

#include <tvm/runtime/registry.h>

#include <cctype>
#include <string>

namespace tvm {
namespace relay {
namespace backend {

std::string ToCFunctionStyle(const std::string& original_name) {
ICHECK(!original_name.empty()) << "Function name is empty";
ICHECK_EQ(original_name.find("TVM"), 0) << "Function not TVM prefixed";

int tvm_prefix_length = 3;
std::string function_name("TVM");

bool new_block = true;
for (const char& symbol : original_name.substr(tvm_prefix_length)) {
Mousius marked this conversation as resolved.
Show resolved Hide resolved
if (std::isalpha(symbol)) {
if (new_block) {
function_name.push_back(std::toupper(symbol));
new_block = false;
} else {
function_name.push_back(std::tolower(symbol));
}
} else if (symbol == '_') {
new_block = true;
}
}
return function_name;
}

std::string ToCVariableStyle(const std::string& original_name) {
ICHECK(!original_name.empty()) << "Variable name is empty";
ICHECK_EQ(original_name.find("TVM"), 0) << "Variable not TVM prefixed";

std::string variable_name;
variable_name.resize(original_name.size());

std::transform(original_name.begin(), original_name.end(), variable_name.begin(), ::tolower);
return variable_name;
}

std::string CombineNames(const Array<String>& names) {
std::stringstream combine_stream;
ICHECK(!names.empty()) << "Name segments empty";

for (const String& name : names) {
ICHECK(!name.empty()) << "Name segment is empty";
combine_stream << name << "_";
}

std::string combined_name = combine_stream.str();
combined_name.pop_back();
return combined_name;
}

std::string SanitizeName(const std::string& name) {
ICHECK(!name.empty()) << "Name is empty";

auto multipleSeparators = [](char before, char after) {
return before == '_' && before == after;
};
auto isNotAlnum = [](char c) { return !std::isalnum(c); };
std::string sanitized_input = name;
std::replace_if(sanitized_input.begin(), sanitized_input.end(), isNotAlnum, '_');

sanitized_input.erase(
std::unique(sanitized_input.begin(), sanitized_input.end(), multipleSeparators),
sanitized_input.end());

return sanitized_input;
}

TVM_REGISTER_GLOBAL("relay.backend.ToCFunctionStyle").set_body_typed(ToCFunctionStyle);
TVM_REGISTER_GLOBAL("relay.backend.ToCVariableStyle").set_body_typed(ToCVariableStyle);
TVM_REGISTER_GLOBAL("relay.backend.PrefixName").set_body_typed(PrefixName);
TVM_REGISTER_GLOBAL("relay.backend.PrefixGeneratedName").set_body_typed(PrefixGeneratedName);
TVM_REGISTER_GLOBAL("relay.backend.SanitizeName").set_body_typed(SanitizeName);

} // namespace backend
} // namespace relay
} // namespace tvm
105 changes: 105 additions & 0 deletions src/relay/backend/name_transforms.h
Original file line number Diff line number Diff line change
@@ -0,0 +1,105 @@
/*
* Licensed to the Apache Software Foundation (ASF) under one
* or more contributor license agreements. See the NOTICE file
* distributed with this work for additional information
* regarding copyright ownership. The ASF licenses this file
* to you 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.
*/

/*!
* \file relay/backend/name_transforms.h
* \brief Transformations which are applied on names to generate appropriately named compiler
* artifacts
*
* Example:
* ToCFunctionStyle(PrefixName(CombineNames({"Device", "target", "Invoke"})))
* // TVMDeviceTargetInvoke
*
* ToCFunctionStyle(PrefixGeneratedName(CombineNames({"model", "Run"})))
* // TVMGenModelRun
*
* ToCVariableStyle(PrefixName(CombineNames({"Device", "target", "t"})))
* // tvm_device_target_t
*
* ToCVariableStyle(PrefixGeneratedName(CombineNames({"model", "Devices"})))
* // tvmgen_model_devices
*
*/

#include <tvm/runtime/container/array.h>
#include <tvm/runtime/container/string.h>
#include <tvm/runtime/logging.h>

#include <algorithm>
#include <iostream>
#include <string>

#ifndef TVM_RELAY_BACKEND_NAME_TRANSFORMS_H_
#define TVM_RELAY_BACKEND_NAME_TRANSFORMS_H_

namespace tvm {
namespace relay {
namespace backend {

/*!
* \brief Transform a name to the C variable style assuming it is
* appropriately constructed using the prefixing functions
* \param original_name Original name
* \return Transformed function in the C function style
*/
std::string ToCFunctionStyle(const std::string& original_name);
areusch marked this conversation as resolved.
Show resolved Hide resolved

/*!
* \brief Transform a name to the C variable style assuming it is
* appropriately constructed using the prefixing functions
* \param name Original name
* \return Transformed function in the C variable style
*/
std::string ToCVariableStyle(const std::string& original_name);

/*!
* \brief Combine names together for use as a generated name
* \param names Vector of strings to combine
* \return Combined together names
*/
std::string CombineNames(const Array<String>& names);

/*!
* \brief Apply TVM-specific prefix to a name
* \param names Vector of names to combine to form a combined name
* \return Name with prefix applied or prefix-only if no name passed
*/
inline std::string PrefixName(const Array<String>& names) { return "TVM_" + CombineNames(names); }

/*!
* \brief Apply generated TVM-specific prefix to a name
* \param names Vector of names to combine to form a combined name
* \return Name with prefix applied or prefix-only if no name passed
*/
inline std::string PrefixGeneratedName(const Array<String>& names) {
return "TVMGen_" + CombineNames(names);
}

/*!
* \brief Sanitize name for output into compiler artifacts
* \param name Original name
* \return Sanitized name
*/
std::string SanitizeName(const std::string& name);

} // namespace backend
} // namespace relay
} // namespace tvm

#endif // TVM_RELAY_BACKEND_NAME_TRANSFORMS_H_
87 changes: 87 additions & 0 deletions tests/cpp/name_transforms_test.cc
Original file line number Diff line number Diff line change
@@ -0,0 +1,87 @@
/*
* Licensed to the Apache Software Foundation (ASF) under one
* or more contributor license agreements. See the NOTICE file
* distributed with this work for additional information
* regarding copyright ownership. The ASF licenses this file
* to you 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.
*/

#include "../src/relay/backend/name_transforms.h"

#include <gtest/gtest.h>
#include <tvm/runtime/container/string.h>

using namespace tvm::relay::backend;
using namespace tvm::runtime;

TEST(NameTransforms, ToCFunctionStyle) {
ASSERT_EQ(ToCFunctionStyle("TVM_Woof"), "TVMWoof");
ASSERT_EQ(ToCFunctionStyle("TVM_woof"), "TVMWoof");
ASSERT_EQ(ToCFunctionStyle("TVM_woof_woof"), "TVMWoofWoof");
ASSERT_EQ(ToCFunctionStyle("TVMGen_woof_woof"), "TVMGenWoofWoof");
EXPECT_THROW(ToCVariableStyle("Cake_Bakery"), InternalError); // Incorrect prefix
EXPECT_THROW(ToCFunctionStyle(""), InternalError);
}

TEST(NameTransforms, ToCVariableStyle) {
ASSERT_EQ(ToCVariableStyle("TVM_Woof"), "tvm_woof");
ASSERT_EQ(ToCVariableStyle("TVM_woof"), "tvm_woof");
ASSERT_EQ(ToCVariableStyle("TVM_woof_Woof"), "tvm_woof_woof");
EXPECT_THROW(ToCVariableStyle("Cake_Bakery"), InternalError); // Incorrect prefix
EXPECT_THROW(ToCVariableStyle(""), InternalError);
}

TEST(NameTransforms, PrefixName) {
ASSERT_EQ(PrefixName({"Woof"}), "TVM_Woof");
ASSERT_EQ(PrefixName({"woof"}), "TVM_woof");
ASSERT_EQ(PrefixName({"woof", "moo"}), "TVM_woof_moo");
EXPECT_THROW(PrefixName({}), InternalError);
EXPECT_THROW(PrefixName({""}), InternalError);
}

TEST(NameTransforms, PrefixGeneratedName) {
ASSERT_EQ(PrefixGeneratedName({"Woof"}), "TVMGen_Woof");
ASSERT_EQ(PrefixGeneratedName({"woof"}), "TVMGen_woof");
ASSERT_EQ(PrefixGeneratedName({"woof", "moo"}), "TVMGen_woof_moo");
EXPECT_THROW(PrefixGeneratedName({}), InternalError);
EXPECT_THROW(PrefixGeneratedName({""}), InternalError);
}

TEST(NameTransforms, CombineNames) {
ASSERT_EQ(CombineNames({"woof"}), "woof");
ASSERT_EQ(CombineNames({"Woof", "woof"}), "Woof_woof");
ASSERT_EQ(CombineNames({"Woof", "woof", "woof"}), "Woof_woof_woof");
ASSERT_EQ(CombineNames({"Woof", "moo", "t"}), "Woof_moo_t");

EXPECT_THROW(CombineNames({}), InternalError);
EXPECT_THROW(CombineNames({""}), InternalError);
EXPECT_THROW(CombineNames({"Woof", ""}), InternalError);
EXPECT_THROW(CombineNames({"", "Woof"}), InternalError);
}

TEST(NameTransforms, SanitizeName) {
ASSERT_EQ(SanitizeName("+_+ "), "_");
ASSERT_EQ(SanitizeName("input+"), "input_");
ASSERT_EQ(SanitizeName("input-"), "input_");
ASSERT_EQ(SanitizeName("input++"), "input_");
ASSERT_EQ(SanitizeName("woof:1"), "woof_1");
EXPECT_THROW(SanitizeName(""), InternalError);
}

TEST(NameTransforms, CombinedLogic) {
ASSERT_EQ(ToCFunctionStyle(PrefixName({"Device", "target", "Invoke"})), "TVMDeviceTargetInvoke");
ASSERT_EQ(ToCFunctionStyle(PrefixGeneratedName({"model", "Run"})), "TVMGenModelRun");
ASSERT_EQ(ToCVariableStyle(PrefixName({"Device", "target", "t"})), "tvm_device_target_t");
ASSERT_EQ(ToCVariableStyle(PrefixGeneratedName({"model", "Devices"})), "tvmgen_model_devices");
}
Loading