https://github.com/Sirraide approved this pull request.
LGTM. `ParseExpressionList()` is used in a lot of places, but given that almost all the other tests seem to have no problem with this change (which honestly surprises me a bit), I feel like this is reasonable (in the tests that are affected, we gain a bogus diagnostic in one and lose one in another, so imo that evens out ;Þ), and a more accurate AST in is always nice. https://github.com/llvm/llvm-project/pull/110133 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
