message: Remove CHECKs that prevent code to be merged
diff: @@ -126,22 +126,11 @@ class AttrBuilder {
  T&& value) const {
  DCHECK(!node_def_finalized_)
  << "Calling SetInAttrValueMap after BuildNodeDef.";
  - // Copied from NodeDefBuilder::Attr
  - const AttrValue* found = AttrSlice(m).Find(attr_name);
  + // If attribute is set more than once, its first value prevails
  + if (AttrSlice(m).Find(attr_name) == nullptr) {
  AttrValue attr_value;
  SetAttrValue(value, &attr_value);
  - if (found == nullptr) {
  m->insert(AttrValueMap::value_type(attr_name, attr_value));
  - } else {
  - // TODO(klessard): remove this validation once automatic input attributes
  - // inference of the C API is used by all clients, or at least Python.
  - // We use it only to verify that attributes inferred have the same values
  - // as those currently added manually (see
  - // github.com/tensorflow/tensorflow/pull/23468#issuecomment-438484058).
  - CHECK_EQ(found->i(), attr_value.i()) << " Number attribute \""
  - << attr_name << "\" was set multiple times with different values";
  - CHECK_EQ(found->type(), attr_value.type()) << " Type attribute \""
  - << attr_name << "\" was set multiple times with different values";
  }
  }
  