fix parallel do hard coded empty var name (#8469)

Yang Yang(Tony) · web-flow · commit 3c2cafbe1d04 · 2018-02-20T15:31:31.000-08:00
diff --git a/paddle/fluid/operators/parallel_do_op.cc b/paddle/fluid/operators/parallel_do_op.cc
@@ -256,7 +256,7 @@ class ParallelDoGradOp : public framework::OperatorBase {
       }
     }
     for (auto &s : Outputs(framework::GradVarName(kParameters))) {
-      if (s == "@EMPTY@") {
+      if (s == framework::kEmptyVarName) {
         continue;
       }
       VLOG(3) << "Moving " << s;
@@ -270,7 +270,7 @@ class ParallelDoGradOp : public framework::OperatorBase {
                       const std::vector<framework::Scope *> &sub_scopes,
                       const platform::PlaceList &places) const {
     for (auto &s : Outputs(framework::GradVarName(kParameters))) {
-      if (s == "@EMPTY@") {
+      if (s == framework::kEmptyVarName) {
         continue;
       }
       VLOG(3) << "Accumulating " << s;

Original file line number	Diff line number	Diff line change
`@@ -256,7 +256,7 @@ class ParallelDoGradOp : public framework::OperatorBase {`
`256`	`256`	`}`
`257`	`257`	`}`
`258`	`258`	`for (auto &s : Outputs(framework::GradVarName(kParameters))) {`
`259`		`- if (s == "@EMPTY@") {`
	`259`	`+ if (s == framework::kEmptyVarName) {`
`260`	`260`	`continue;`
`261`	`261`	`}`
`262`	`262`	`VLOG(3) << "Moving " << s;`
`@@ -270,7 +270,7 @@ class ParallelDoGradOp : public framework::OperatorBase {`
`270`	`270`	`const std::vector<framework::Scope *> &sub_scopes,`
`271`	`271`	`const platform::PlaceList &places) const {`
`272`	`272`	`for (auto &s : Outputs(framework::GradVarName(kParameters))) {`
`273`		`- if (s == "@EMPTY@") {`
	`273`	`+ if (s == framework::kEmptyVarName) {`
`274`	`274`	`continue;`
`275`	`275`	`}`
`276`	`276`	`VLOG(3) << "Accumulating " << s;`