[PATCH] D122083: [Concepts] Fix placeholder constraints when references are involved

Roy Jacobson via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Mar 23 06:55:32 PDT 2022


royjacobson marked an inline comment as done.
royjacobson added a comment.

In D122083#3402289 <https://reviews.llvm.org/D122083#3402289>, @erichkeane wrote:

> Can you add some tests for the OTHER forms of 'auto' as well?  We have `decltype(auto)` and `auto_type`, and I want to make sure whatever we do with those 'looks right'.

Can we have constraints on `__auto_type`? As far as I understand it, it's a C extension with very limited C++ support.
About decltype(auto) - we can't have `*`/`&` modifiers with it, and that's already covered by tests like p7-cxx14.

So (I think?) it's always invalid code and it fails earlier during parsing.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D122083



More information about the cfe-commits mailing list