sammccall added a comment. I think per offline discussion, this was going to have some additional docs clarifying the contracts of this function, and maybe extra tests?
The conclusion IIRC was that this function is designed to find the text that you could replace in order to replace an AST node, so it's fairly conservative (but arg expansions are fair game assuming expanded once). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D75446/new/ https://reviews.llvm.org/D75446 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
