LegalizeAdulthood added a comment. In D124650#3481514 <https://reviews.llvm.org/D124650#3481514>, @LegalizeAdulthood wrote:
> General question: > > There is the question of whether or not the replacement should have `()`s > around the whole > thing because we don't analyze the context to see if the change in precedence > between > `operator!` and `operator&&` or `operator||` makes any difference. I suppose it's better to be safe than sorry, so I'll update the check to surround the replacement with parens. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D124650/new/ https://reviews.llvm.org/D124650 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits