[flang-commits] [flang] [flang] Use llvm::any_of and llvm::none_of (NFC) (PR #102797)

Kazu Hirata via flang-commits flang-commits at lists.llvm.org
Sat Aug 17 18:04:21 PDT 2024


kazutakahirata wrote:

> Why is this a good change? Replacing standard C++ with local idioms doesn't improve readability.

The change avoids repeating the same container names multiple times.  Your comment about the local idiom is well taken, but we can also look at it as using `std::ranges::none_of` ahead of time before C++20 is available in our codebase.  Many utility functions in `llvm/include/llvm/ADT/{STLExtras.h,STLForwardCompat.h,bit.h}` offer such functionality.


https://github.com/llvm/llvm-project/pull/102797


More information about the flang-commits mailing list