serge-sans-paille added a comment.

(keeping the ball rolling) lYou want to exploit the presence of a sentinel in 
the domain of values of a given type. Maybe make that explicit through the 
method naming instead of using the `isValid` theme.

Some bibliography on the subject: 
https://groups.google.com/a/isocpp.org/forum/#!topic/std-proposals/46J1onhWJ-s/discussion.

On a personal note, as much as I like the idea (and I really do like it), I'd 
rather have llvm::optional sticking to std::optional interface.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D69230



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

Reply via email to