From 75c7bca740935a0cca462e28475dd6b046a6872c Mon Sep 17 00:00:00 2001 From: Sergei Barannikov Date: Tue, 13 Aug 2024 04:00:19 +0300 Subject: [DataLayout] Remove constructor accepting a pointer to Module (#102841) The constructor initializes `*this` with `M->getDataLayout()`, which is effectively the same as calling the copy constructor. There does not seem to be a case where a copy would be necessary. Pull Request: https://github.com/llvm/llvm-project/pull/102841 --- llvm/lib/Transforms/Utils/InlineFunction.cpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'llvm/lib/Transforms/Utils/InlineFunction.cpp') diff --git a/llvm/lib/Transforms/Utils/InlineFunction.cpp b/llvm/lib/Transforms/Utils/InlineFunction.cpp index 9c9fc7a..94e8765 100644 --- a/llvm/lib/Transforms/Utils/InlineFunction.cpp +++ b/llvm/lib/Transforms/Utils/InlineFunction.cpp @@ -1896,8 +1896,8 @@ static void trackInlinedStores(Function::iterator Start, Function::iterator End, LLVM_DEBUG(errs() << "trackInlinedStores into " << Start->getParent()->getName() << " from " << CB.getCalledFunction()->getName() << "\n"); - std::unique_ptr DL = std::make_unique(CB.getModule()); - at::trackAssignments(Start, End, collectEscapedLocals(*DL, CB), *DL); + const DataLayout &DL = CB.getDataLayout(); + at::trackAssignments(Start, End, collectEscapedLocals(DL, CB), DL); } /// Update inlined instructions' DIAssignID metadata. We need to do this -- cgit v1.1