https://github.com/shafik requested changes to this pull request.
It looks like we are using the same pattern that fix is using in `ASTStmtReader::VisitRequiresExpr` which gives me some comfort but I am not happy that we have a relatively complex and easy to get wrong operation repeated in three places now. This feel like it should be refactored into common code and if we find a bug w/ it later on fixing it one place will fix all the places. https://github.com/llvm/llvm-project/pull/98699 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
