================
@@ -79,14 +79,16 @@ struct ValueMapConfig {
   static mutex_type *getMutex(const ExtraDataT &/*Data*/) { return nullptr; }
 };
 
+/// This type stores Metadata. Used in ValueMap.
+using MDMapT = DenseMap<const Metadata *, TrackingMDRef>;
----------------
felipepiovezan wrote:

I don't think we should have a type alias that is an implementation detail in 
the `llvm::` namespace, as it adds visibility to this everywhere. We should 
instead move it to the public `ValueMap` API and refer to it as 
`ValueMap::MDMapT` (or just spell out the type completely and not worry about 
the typedef)

https://github.com/llvm/llvm-project/pull/118622
_______________________________________________
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits

Reply via email to