felix642 wrote: LGTM ! You'll need to update your branch with master but the PR looks good.
The only thing that seems odd is the changes that you've made to the method `areParensNeededForOverloadedOperator` where the Operators `OO_New`, `OO_Delete`, `OO_Array_New` and `OO_ArrayDelete` now need parenthesis where they did not before. I'm assuming that you have a used case in mind where parenthesis would be needed and this is why you've changed the behaviour. https://github.com/llvm/llvm-project/pull/74891 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits