================ @@ -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