================ @@ -375,10 +390,48 @@ getFunctionKeyImpl(uint32_t ParentContextID, llvm::StringRef Name, return FunctionTableKey(ParentContextID, *NameID); } +template <typename GetIdentifierFn> ---------------- j-hui wrote:
Does this need to be generic over `GetIdentifierFn`? Isn't its type signature fixed to `llvm:function_ref<std::optional<IdentifierID>(llvm::StringRef)>`? Same with the other overload of `getFunctionKeyImpl` https://github.com/llvm/llvm-project/pull/216148 _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
