aaron.ballman accepted this revision. aaron.ballman added a comment. This revision is now accepted and ready to land.
LGTM assuming there are no surprises from the additional test request. ================ Comment at: clang/test/AST/Interp/literals.cpp:762 + }; + static_assert(!__is_trivial(S2), ""); +} ---------------- I'd like to see a test where the argument is value dependent: https://godbolt.org/z/a1W68vhh7 Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D142448/new/ https://reviews.llvm.org/D142448 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits