ahatanak added inline comments.

================
Comment at: include/clang/Basic/DiagnosticSemaKinds.td:633
+  "capture a variable of type %1}3 "
+  "since it %select{contains a union that |}2is non-trivial to "
+  "%select{default-initialize|destruct|copy}0">;
----------------
rjmccall wrote:
> Should this be "`%select{contains|is}2 a union that is non-trivial to..."?  
> None of these situations bans non-trivial types, just unions containing them 
> (and aggregates containing such unions).
Yes, that's correct. We only ban aggregates containing non-trivial unions (the 
former case) and non-trivial unions (the latter case).


Repository:
  rC Clang

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

https://reviews.llvm.org/D63753



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

Reply via email to