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
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
