ilya-biryukov added a comment.

I don't think it's unreasonable to protect against `nullptr` in individual 
fields. We need **some** value for it and null is a reasonable choice.
However, it feels completely wrong to have `nullptr` in collections. Those 
should be filtered out upon creation, if possible.
It seems perfectly fine for a collection to have less elements if some of them 
are invalid.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D135257

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

Reply via email to