tahonermann accepted this revision.
tahonermann added a comment.
This revision is now accepted and ready to land.

Thanks @Manna, this looks good to me now!



================
Comment at: clang/include/clang/Sema/ParsedAttr.h:705-707
+  // The move assignment operator is defined as deleted pending further
+  // motivation.
+  AttributePool &operator=(AttributePool &&pool) = delete;
----------------
Manna wrote:
> tahonermann wrote:
> > Let's explicitly delete the copy assignment operator too since the copy 
> > constructor is deleted.
> @tahonermann , copy assignment operator has already been explicitly deleted 
> at line no 701. I made this change part of https://reviews.llvm.org/D149718. 
> It was a rebase issue, so it did show it before with my previous patch. I 
> have fixed it now and added comment since the copy assignment was added 
> explicitly as deleted.
Ah, perfect!


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D150411/new/

https://reviews.llvm.org/D150411

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to