LegalizeAdulthood added a comment.

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.


Repository:
  rG LLVM Github Monorepo

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

Reply via email to