areusch commented on code in PR #12066: URL: https://github.com/apache/tvm/pull/12066#discussion_r931619236
########## include/tvm/ir/name_supply.h: ########## @@ -0,0 +1,139 @@ +/* + * 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 tvm/ir/name_supply.h + * \brief NameSupply that can be used to generate unique variable names. + */ +#ifndef TVM_IR_NAME_SUPPLY_H_ +#define TVM_IR_NAME_SUPPLY_H_ + +#include <string> +#include <unordered_map> + +#include "tvm/ir/expr.h" + +namespace tvm { + +/*! + * \brief NameSupply can be used to generate unique names. + */ +class NameSupplyNode : public Object { + public: + /*! + * \brief Empty constructor. Will use an empty prefix for the constructed NameSupply. + */ + NameSupplyNode() : NameSupplyNode("") {} + + /*! + * \brief Constructs a NameSupply with the given \p prefix. + */ + explicit NameSupplyNode(const String& prefix); + + /*! + * \brief Generates a unique name from this NameSupply. + * \param name The name from which the generated name is derived. + * \param add_prefix If set to true, then the prefix of this NameSupply will be prepended to the + * name. \return A unique name. + */ + String FreshName(const String& name, bool add_prefix = true); + + /*! + * \brief Reserves an existing name with this NameSupply. + * \param name The name to be reserved. + * \param add_prefix If set to true, then the prefix of this NameSupply will be prepended to the + * name before reserving it. \return The name that was reserved with the NameSupply. It can be + * different if a prefix is added. + */ + String ReserveName(const String& name, bool add_prefix = true); + + /*! + * \brief Checks if this NameSupply already generated a name. + * \param name The name to check. + * \param add_prefix If set to true, then the prefix of this NameSupply will be prepended to the + * name before checking for it. \return True if the name has already been generated. False + * otherwise. + */ + bool ContainsName(const String& name, bool add_prefix = true); + + /*! + * \brief Clears this NameSupply of all the generated names. + */ + void Clear(); + + void VisitAttrs(AttrVisitor* v) { v->Visit("prefix", &prefix_); } Review Comment: i think you can omit it if you don't want ppl to modify `prefix_`. Since prefix is private in C++, we should not expose it through reflection here. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@tvm.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org