experiments/STT-tensorflow
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Add a "Clear" method in FunctionLibraryDefinition.

Browse Source 
PiperOrigin-RevId: 328682257
Change-Id: Ib1db2ed042f30218f7d6ced101561d07774300be
This commit is contained in:
Jing Pu 2020-08-26 23:25:02 -07:00 committed by TensorFlower Gardener
parent c64796cbba
commit c5e13d84ca
3 changed files with 19 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
tensorflow/core/framework/function.cc
View File

@ -1454,6 +1454,12 @@ Status FunctionLibraryDefinition::RemoveFunctionHelper(const string& func) {
return Status::OK(); return Status::OK();
} }
void FunctionLibraryDefinition::Clear() {
mutex_lock l(mu_);
function_defs_.clear();
func_grad_.clear();
}
Status FunctionLibraryDefinition::RemoveGradient(const string& func) { Status FunctionLibraryDefinition::RemoveGradient(const string& func) {
const auto& i = func_grad_.find(func); const auto& i = func_grad_.find(func);
if (i == func_grad_.end()) { if (i == func_grad_.end()) {

3
tensorflow/core/framework/function.h
View File

@ -403,6 +403,9 @@ class FunctionLibraryDefinition : public OpRegistryInterface {
// are no longer in use. // are no longer in use.
Status RemoveFunction(const std::string& func) TF_LOCKS_EXCLUDED(mu_); Status RemoveFunction(const std::string& func) TF_LOCKS_EXCLUDED(mu_);
// Removes all the functions and gradient functions.
void Clear() TF_LOCKS_EXCLUDED(mu_);
// Adds the functions and gradients in 'other' to this function library. // Adds the functions and gradients in 'other' to this function library.
// Duplicate functions and gradients are ignored. // Duplicate functions and gradients are ignored.
// This operation is atomic. // This operation is atomic.

10
tensorflow/core/framework/function_test.cc
View File

@ -1068,6 +1068,16 @@ TEST(FunctionLibraryDefinitionTest, RemoveFunction) {
EXPECT_FALSE(lib_def.Contains("XTimesTwo")); EXPECT_FALSE(lib_def.Contains("XTimesTwo"));
} }
TEST(FunctionLibraryDefinitionTest, Clear) {
FunctionLibraryDefinition lib_def(OpRegistry::Global(), {});
TF_CHECK_OK(lib_def.AddFunctionDef(test::function::XTimesTwo()));
TF_CHECK_OK(lib_def.AddFunctionDef(test::function::XAddX()));
lib_def.Clear();
EXPECT_FALSE(lib_def.Contains("XTimesTwo"));
EXPECT_FALSE(lib_def.Contains("XAddX"));
}
TEST(FunctionLibraryDefinitionTest, AddLibrary) { TEST(FunctionLibraryDefinitionTest, AddLibrary) {
// Create lib def with single function // Create lib def with single function
FunctionDefLibrary proto; FunctionDefLibrary proto;