diff options
author | Skye Wanderman-Milne <skyewm@google.com> | 2017-05-01 07:56:28 -0800 |
---|---|---|
committer | TensorFlower Gardener <gardener@tensorflow.org> | 2017-05-01 09:04:29 -0700 |
commit | 474aba363a85d68fdd8f8ab8b1d9d749a91e6e7a (patch) | |
tree | f0df306b0434e8079408659224c4c7bfce8ced20 /tensorflow/core/graph/graph.cc | |
parent | 2a1a0d729fb23ac8758fff64119503e8e3e7b94d (diff) |
Make FunctionLibraryDefinition::AddFunctionDef() check for conflicting op name
This prevents a function from masking an existing op.
Change: 154720287
Diffstat (limited to 'tensorflow/core/graph/graph.cc')
-rw-r--r-- | tensorflow/core/graph/graph.cc | 7 |
1 files changed, 0 insertions, 7 deletions
diff --git a/tensorflow/core/graph/graph.cc b/tensorflow/core/graph/graph.cc index fae9f26f76..e1657cb862 100644 --- a/tensorflow/core/graph/graph.cc +++ b/tensorflow/core/graph/graph.cc @@ -380,13 +380,6 @@ Status Graph::AddFunctionLibrary(const FunctionDefLibrary& fdef_lib) { // Ignore duplicate FunctionDefs continue; } - // TODO(skyewm): fix test breakages and reenable this check - // const OpDef* op_def; - // if (ops_.LookUpOpDef(fdef.signature().name(), &op_def).ok()) { - // return errors::InvalidArgument( - // "Cannot add function '", fdef.signature().name(), - // "' because an op with the same name already exists."); - // } TF_RETURN_IF_ERROR(ops_.AddFunctionDef(fdef)); } for (const GradientDef& grad : fdef_lib.gradient()) { |