tbaeder added inline comments.
================ Comment at: clang/test/AST/Interp/builtin-functions.cpp:153 + static_assert(__builtin_LINE() == 152, ""); +} ---------------- 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. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D155627/new/ https://reviews.llvm.org/D155627 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
