[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