Sirraide wrote: Update: Iterating expansion statements require building a lambda in Sema (see part 5 in this patch series), which necessitated moving `ParseScope` and friends from the Parser into Sema; I’ve included this refactor in this NFC patch.
As an aside, it makes more sense for `ParseScope` etc. to be in Sema anyway since its implementation exclusively calls Sema functions—technically, it does call _some_ parser functions, but _those functions_ then in turn only call Sema functions and don’t actually access any parser state. The only reference to parser state is that the current token location is passed to Sema... but Sema doesn’t do anything w/ it, so I just removed that parameter. https://github.com/llvm/llvm-project/pull/169683 _______________________________________________ llvm-branch-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
