jlebar added a comment.

> Looks right to me (other than the missing constexpr in C++14 onwards). Though 
> this is subtle enough that I suspect the only way to know for sure is to try 
> it.

Thanks a lot, Richard.  FTR the missing constexpr is in 
https://reviews.llvm.org/D48151.


https://reviews.llvm.org/D48036



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
  • [PATCH] D48036: [... Justin Lebar via Phabricator via cfe-commits
    • [PATCH] D480... Richard Smith - zygoloid via Phabricator via cfe-commits
    • [PATCH] D480... Justin Lebar via Phabricator via cfe-commits
    • [PATCH] D480... Justin Lebar via Phabricator via cfe-commits

Reply via email to