aaron.ballman added a reviewer: rsmith. aaron.ballman added inline comments.
================ Comment at: clang/lib/Sema/SemaTemplate.cpp:3598 return true; + } else if (auto *UE = dyn_cast<UnaryExprOrTypeTraitExpr>(E)) { + Expr::EvalResult Result; ---------------- ================ Comment at: clang/test/SemaCXX/static-assert.cpp:218 +// expected-note@-2{{with 'alignof(IntAndPointer)' equal to 8}} +// expected-note@-3{{with 'sizeof(IntAndPointer)' equal to 16}} ---------------- Can you also add a test where the unary expression isn't the reason for the failure? e.g., ``` static_assert(is_same_v<float, decltype(sizeof(int))>); ``` Perhaps we'd want to silence the notes if the expression context for the call to `sizeof` is itself unevaluated? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D108211/new/ https://reviews.llvm.org/D108211 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits