New check suggestion: use LLVM STLExtras range-based API

[kirillbobyrev]

Bugzilla Link	39120
Version	unspecified
OS	All
CC	@JonasToth

Extended Description

LLVM has include/llvm/STLExtras.h full of Standard Library function replacements with more convenient API leveraging ranges, such as

void llvm::sort (Container &&C)
void llvm::sort (Container &&C, Compare Comp)
auto llvm::find (R &&Range, const T &Val) -> decltype(adl_begin(Range))
UnaryPredicate llvm::for_each (R &&Range, UnaryPredicate P)
...

There are several patches with manual cleanups (D52576, D52650) and it would be nice to

• automatically detect these cases
• enforce readability by providing diagnostics later on

[JonasToth]

Could be generalized to a modernize-use-ranges transforming check. There are other range libraries (v3) and the standardization effort that should be considered

[5chmidti]

modernize-use-ranges has been implemented in #97764. It currently only supports the STL and boost (through boost-use-ranges)

[kirillbobyrev]

modernize-use-ranges has been implemented in #97764. It currently only supports the STL and boost (through boost-use-ranges)

Thanks for noticing!

[5chmidti]

The above check is not actually yet implementing using LLVM's ranges algorithms, which would be nice to have as I am not sure when we will switch to C++20