[PATCH] D156045: [clang][Interp] Enable existing source_location tests

[Corentin Jabot via Phabricator via cfe-commits]

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