[llvm] [GC] Use `MapVector` for `GCStrategyMap` (PR #132729)

[Philip Reames via llvm-commits]

================
@@ -151,9 +152,9 @@ class GCFunctionInfo {
   size_t live_size(const iterator &p) const { return roots_size(); }
 };
 
-struct GCStrategyMap {
-  StringMap<std::unique_ptr<GCStrategy>> StrategyMap;
-
+class GCStrategyMap : public MapVector<std::string, std::unique_ptr<GCStrategy>,
----------------
preames wrote:

Please, do not inherit from core data structures.  Contain them instead.

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