https://github.com/gamesh411 approved this pull request.
This is a nice cleanup change. I have checked the test cases and the reasoning behind the removals. It is also welcome that these reasons are documented, at least in the PR. @NagyDonat What do you think about making these reasoning snippets available in the commit message? I am not sure that it is enough for them to only exist in the PR comments, but if you say that it would be unnecessary verbose to have the gist of them in the commit message, that is fine as well. https://github.com/llvm/llvm-project/pull/128508 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
