[PATCH] D146061: [ADT] Make llvm::is_contained call member `contains` or `find` when available

Kazu Hirata via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 15 08:54:58 PDT 2023


kazu accepted this revision.
kazu added a comment.

LGTM.  I like the fallback mechanism.  We prefer to use `X.contains()`, but if that's not possible (or people just randomly call `llvm::is_contained`), `llvm::is_contained` is there to do the right thing for you.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D146061



More information about the llvm-commits mailing list