[RFC] Implicit options from view with underlying copy

This seems non-surprising to me but better RFC. Comments?

Author: nicolasvasilache

include/tc/core/mapping_options-inl.h

@@ -243,34 +243,29 @@ MappingOptions MappingOptions::makeUnmappedMappingOptions() {
 }
 
 MappingOptions MappingOptions::makeNaiveMappingOptions() {
-  return MappingOptions(
-      makeUnmappedMappingOptions().view.tile({32, 32, 32}).unroll(1));
+  return makeUnmappedMappingOptions().view.tile({32, 32, 32}).unroll(1);
 }
 
 MappingOptions MappingOptions::makeSingleThreadMappingOptions() {
-  return MappingOptions(makeUnmappedMappingOptions().view.tile({1}).unroll(1));
+  return makeUnmappedMappingOptions().view.tile({1}).unroll(1);
 }
 
 MappingOptions MappingOptions::makePointwiseMappingOptions() {
-  return MappingOptions(
-      makeUnmappedMappingOptions().view.tile({32, 32, 32}).unroll(128));
+  return makeUnmappedMappingOptions().view.tile({32, 32, 32}).unroll(128);
 }
 
 MappingOptions MappingOptions::makeMlpMappingOptions() {
-  return MappingOptions(
-      makeUnmappedMappingOptions()
-          .view.outerScheduleFusionStrategy(FusionStrategy::Max)
-          .tile({1})
-          .unroll(1));
+  return makeUnmappedMappingOptions()
+      .view.outerScheduleFusionStrategy(FusionStrategy::Max)
+      .tile({1})
+      .unroll(1);
 }
 
 MappingOptions MappingOptions::makeConvolutionMappingOptions() {
-  return MappingOptions(
-      makeUnmappedMappingOptions().view.tile({4, 8, 8, 8}).unroll(1));
+  return makeUnmappedMappingOptions().view.tile({4, 8, 8, 8}).unroll(1);
 }
 
 MappingOptions MappingOptions::makeGroupConvolutionMappingOptions() {
-  return MappingOptions(
-      makeUnmappedMappingOptions().view.tile({1, 1}).unroll(1));
+  return makeUnmappedMappingOptions().view.tile({1, 1}).unroll(1);
 }
 } // namespace tc

include/tc/core/mapping_options.h

@@ -315,7 +315,7 @@ class MappingOptions {
   explicit MappingOptions(const MappingOptionsProto& proto)
       : ownedProto_(proto), view(ownedProto_) {}
   /// Performs an underlying copy of the proto viewed by ```view```
-  explicit MappingOptions(const MappingOptionsView& view)
+  /* implicit */ MappingOptions(const MappingOptionsView& view)
       : ownedProto_(view.proto), view(ownedProto_) {}
 
   std::string toProtobufSerializedString() const {