tra added a comment. In D61458#1488550 <https://reviews.llvm.org/D61458#1488550>, @hliao wrote:
> In D61458#1488523 <https://reviews.llvm.org/D61458#1488523>, @tra wrote: > > > Perhaps we should allow this in all unevaluated contexts? > > I.e. `int s = sizeof(foo(x));` should also work. > > > good point, do we have a dedicated context for sizeof? that make the checking > easier. Sema::isUnevaluatedContext() may be able to do the job. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D61458/new/ https://reviews.llvm.org/D61458 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits