[llvm-branch-commits] [clang] clang: Avoid intermediate DenseSet of triples (PR #189263)

[Matt Arsenault via llvm-branch-commits]

arsenm wrote:

> These were changed to be a multiset in [99b2a11](https://github.com/llvm/llvm-project/commit/99b2a1143f932c2da8395a69072f177f596547b2) from a vector. I don't think `DenseSet` has a deterministic iteration order, so is this correct? I think the concern is that tests that use more than one toolchain will fail on different machines.

No, this is not a multiset. It's a DenseSet which only exists to insert into the multiset. This just folds to directly use the multiset

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