fixup Release builds

src/Domain/CoordinateMaps/CoordinateMap.tpp

@@ -77,7 +77,14 @@ struct map_type<std::unique_ptr<T>> {
 template <typename T>
 using map_type_t = typename map_type<T>::type;
 
+// For standard compliance, this function definition would have to be
+// available whenever it is called from CoordinateMap::pup (or we'd
+// have to explicitly instantiate it).  In practice, marking it
+// noinline makes the call from the constructor below visible to the
+// linker and puts less stuff in the header file.  If that stops
+// working, it can be moved.
 template <typename... Maps, size_t... Is>
+__attribute__((noinline))
 std::unordered_set<std::string> initialize_names(
     const std::tuple<Maps...>& maps, std::index_sequence<Is...> /*meta*/) {
   std::unordered_set<std::string> function_of_time_names{};