njames93 added a comment. Shouldn't the constructors that need `std::move` also ensure the `<utility>` header is also included, or at least transitively included. We don't want to generate constructors that wont compile.
Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D116514/new/ https://reviews.llvm.org/D116514 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits