[PATCH] D148690: [clang][Interp] Handle __extension__ unary operators

[Timm Bäder via Phabricator via cfe-commits]

tbaeder added inline comments.


================
Comment at: clang/test/AST/Interp/literals.cpp:791
+#endif
     return __FUNCTION__[index];
   }
----------------
aaron.ballman wrote:
> tbaeder wrote:
> > erichkeane wrote:
> > > tbaeder wrote:
> > > > It's weird that the above two statements warn about using `__FUNCTION__` and the return statement doesn't.
> > > Aren't they warning about it being 'unused'?  If so, the 'return' uses them...
> > Right, my message wasn't clear but I meant the c++14 extension diagnostic: https://godbolt.org/z/Kso8qv7jh
> That is pretty strange, tbh. Might be worth investigating in a follow-up.
Yeah, in `CheckConstexprFunctionStmt` in `SemaDeclCXX.cpp`, the return expression is not checked.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D148690