diff --git a/mlir/include/mlir/IR/DialectImplementation.h b/mlir/include/mlir/IR/DialectImplementation.h
--- a/mlir/include/mlir/IR/DialectImplementation.h
+++ b/mlir/include/mlir/IR/DialectImplementation.h
@@ -140,12 +140,18 @@
   }
 };
 
+namespace detail {
+template <typename T>
+using has_push_back_t = decltype(
+    std::declval<T>().push_back(std::declval<typename T::value_type &&>()));
+} // namespace detail
+
 /// Parse any container that supports back insertion as a list.
 template <typename ContainerT>
-struct FieldParser<
-    ContainerT, std::enable_if_t<std::is_member_function_pointer<
-                                     decltype(&ContainerT::push_back)>::value,
-                                 ContainerT>> {
+struct FieldParser<ContainerT,
+                   std::enable_if_t<llvm::is_detected<detail::has_push_back_t,
+                                                      ContainerT>::value,
+                                    ContainerT>> {
   using ElementT = typename ContainerT::value_type;
   static FailureOr<ContainerT> parse(AsmParser &parser) {
     ContainerT elements;
@@ -153,7 +159,7 @@
       auto element = FieldParser<ElementT>::parse(parser);
       if (failed(element))
         return failure();
-      elements.push_back(*element);
+      elements.push_back(std::move(*element));
       return success();
     };
     if (parser.parseCommaSeparatedList(elementParser))