cor3ntin accepted this revision. cor3ntin added inline comments. This revision is now accepted and ready to land.
================ Comment at: clang/test/SemaCXX/source_location.cpp:764-768 +#if defined(USE_CONSTEVAL) && !defined(NEW_INTERP) static_assert(test_init_capture() == __LINE__ - 4); #else static_assert(test_init_capture() == __LINE__ ); #endif ---------------- tbaeder wrote: > The comment block just before this `#if` seems to say that the new > interpreter is correct here...? Yes, which is interesting. Maybe modifying the existing interpreter properly would be doable then? I'll have to look into that... I'm surprised it's only an evaluation issue Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D156045/new/ https://reviews.llvm.org/D156045 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
