llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT--> @llvm/pr-subscribers-lto Author: Artem Pianykh (artempyanykh) <details> <summary>Changes</summary> [NFC] Remove adhoc definition of MDMapT in IRMover Summary: The typedef was there probably because the type alias in ValueMap was private. Test Plan: ninja check-llvm-unit check-llvm --- Full diff: https://github.com/llvm/llvm-project/pull/118626.diff 2 Files Affected: - (modified) llvm/include/llvm/Linker/IRMover.h (+1-3) - (modified) llvm/lib/Linker/IRMover.cpp (-3) ``````````diff diff --git a/llvm/include/llvm/Linker/IRMover.h b/llvm/include/llvm/Linker/IRMover.h index 1e3c5394ffa2af..89e9cbe0be18e1 100644 --- a/llvm/include/llvm/Linker/IRMover.h +++ b/llvm/include/llvm/Linker/IRMover.h @@ -12,6 +12,7 @@ #include "llvm/ADT/ArrayRef.h" #include "llvm/ADT/DenseSet.h" #include "llvm/ADT/FunctionExtras.h" +#include "llvm/IR/ValueMap.h" #include <functional> namespace llvm { @@ -41,9 +42,6 @@ class IRMover { static bool isEqual(const StructType *LHS, const StructType *RHS); }; - /// Type of the Metadata map in \a ValueToValueMapTy. - typedef DenseMap<const Metadata *, TrackingMDRef> MDMapT; - public: class IdentifiedStructTypeSet { // The set of opaque types is the composite module. diff --git a/llvm/lib/Linker/IRMover.cpp b/llvm/lib/Linker/IRMover.cpp index a0c3f2c5b0baf6..76af83df00cba2 100644 --- a/llvm/lib/Linker/IRMover.cpp +++ b/llvm/lib/Linker/IRMover.cpp @@ -373,9 +373,6 @@ class LocalValueMaterializer final : public ValueMaterializer { Value *materialize(Value *V) override; }; -/// Type of the Metadata map in \a ValueToValueMapTy. -typedef DenseMap<const Metadata *, TrackingMDRef> MDMapT; - /// This is responsible for keeping track of the state used for moving data /// from SrcM to DstM. class IRLinker { `````````` </details> https://github.com/llvm/llvm-project/pull/118626 _______________________________________________ llvm-branch-commits mailing list llvm-branch-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
