[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