[PATCH] D72282: [clang-tidy] Add `bugprone-unintended-adl`

[Arthur O'Dwyer via Phabricator via cfe-commits]

Quuxplusone added a comment.

Repeating for emphasis: This is awesome. :)

Having the opt-in //option// to not-ignore overloaded operators is good; please keep that option (in case anyone tries to talk you into removing it).

For more than you wanted to know about ADL, why it's bad for library writers, and why a compiler check is a great idea, see https://quuxplusone.github.io/blog/2019/09/26/uglification-doesnt-stop-adl/ (It ends with a secret bonus test case, where certain unqualified infix operators do //not// use ADL even though they look like they should.)


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D72282/new/

https://reviews.llvm.org/D72282