[llvm] [ADT] Add DenseSet(llvm::from_t, Range) (PR #131832)
Nikita Popov via llvm-commits
llvm-commits at lists.llvm.org
Wed Mar 19 01:51:02 PDT 2025
nikic wrote:
@kuhar I think the way history played out there is that we added to_vector for this use case, but then still ended up adding the ArrayRef overload to SmallVector later because the ergonomics of having the ctor are just better.
I think that if there are no technical issues with it, having a ctor accepting a range is a good idea.
https://github.com/llvm/llvm-project/pull/131832
More information about the llvm-commits
mailing list