[PATCH] D146061: [ADT] Make llvm::is_contained call member `contains` when available
Amir Ayupov via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Mar 14 17:01:06 PDT 2023
Amir added a comment.
I’m fully supportive of this change. BOLT uses this interface which helps maintaibilty as it clearly states the intent and is robust wrt container type change. It’s a clear win to also have the same runtime characteristics as `find`.
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