[PATCH] D155627: [clang][Interp] Handle SourceLocExprs

Wed Jul 19 07:53:10 PDT 2023

cor3ntin added inline comments.

================
Comment at: clang/test/AST/Interp/builtin-functions.cpp:153
+  static_assert(__builtin_LINE() == 152, "");
+}
----------------
tbaeder wrote:
> tbaeder wrote:
> > cor3ntin wrote:
> > > what is missing to enable the existing source location tests with the new intterpreter?
> > There is a crash that I need to investigate first, //but// `SemaCXX/source_location.cpp` also uses stuff like `static_assert(noexcept(...));` that I don't handle yet.
> Okay, after my last two patches I uploaded, the only problem missing is the broken source locations for initializers.
I think it might be worth doing in this patch (unless it requires a lot of work)

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D155627/new/

https://reviews.llvm.org/D155627