diff options
| author | Sergei Barannikov <barannikov88@gmail.com> | 2024-08-13 04:00:19 +0300 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2024-08-13 04:00:19 +0300 |
| commit | 75c7bca740935a0cca462e28475dd6b046a6872c (patch) | |
| tree | 664d10bf0a09d65c386bfc290b3518020f7df399 /llvm/lib/Transforms/Utils/InlineFunction.cpp | |
| parent | f8f34c7e4914ce9e90453e902e2e03f129119fb6 (diff) | |
| download | llvm-75c7bca740935a0cca462e28475dd6b046a6872c.zip llvm-75c7bca740935a0cca462e28475dd6b046a6872c.tar.gz llvm-75c7bca740935a0cca462e28475dd6b046a6872c.tar.bz2 | |
[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
Diffstat (limited to 'llvm/lib/Transforms/Utils/InlineFunction.cpp')
| -rw-r--r-- | llvm/lib/Transforms/Utils/InlineFunction.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
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<DataLayout> DL = std::make_unique<DataLayout>(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 |