mizvekov added inline comments.

================
Comment at: clang/lib/Sema/SemaDeclCXX.cpp:7876
+                  diag::
+                      
err_defaulted_comparison_cannot_deduce_unsupported_builtin_type)
+                  << Subobj.Kind << Subobj.Decl << T;
----------------
I think this should be an error because otherwise we would be accepting and 
giving this program a different semantic just because of some not-implemented 
feature.
But I can see the downside that we could potentially reject a program which is 
not affected by the operator being implicitly deleted.

Thoughts?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D103850

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
  • [PATCH] D103850: ... Matheus Izvekov via Phabricator via cfe-commits
    • [PATCH] D103... Matheus Izvekov via Phabricator via cfe-commits
    • [PATCH] D103... Richard Smith - zygoloid via Phabricator via cfe-commits
    • [PATCH] D103... Matheus Izvekov via Phabricator via cfe-commits

Reply via email to