[all-commits] [llvm/llvm-project] e26b42: [lldb] Avoid calling DataLayout constructor accept...

Sergei Barannikov via All-commits all-commits at lists.llvm.org
Mon Aug 12 11:47:08 PDT 2024


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: e26b42c70d37cbea47016984e1d2bfc347fb8818
      https://github.com/llvm/llvm-project/commit/e26b42c70d37cbea47016984e1d2bfc347fb8818
  Author: Sergei Barannikov <barannikov88 at gmail.com>
  Date:   2024-08-12 (Mon, 12 Aug 2024)

  Changed paths:
    M lldb/source/Expression/IRInterpreter.cpp
    M lldb/source/Plugins/ExpressionParser/Clang/IRDynamicChecks.cpp
    M lldb/source/Plugins/ExpressionParser/Clang/IRForTarget.cpp
    M lldb/source/Plugins/ExpressionParser/Clang/IRForTarget.h

  Log Message:
  -----------
  [lldb] Avoid calling DataLayout constructor accepting Module pointer (NFC) (#102839)

The constructor initializes `*this` with a copy of `M->getDataLayout()`,
which can just be spelled as `DataLayout DL = M->getDataLayout()`. In
all places where the constructor is used, Module outlives DataLayout, so
store a reference to it instead of cloning.

Pull Request: https://github.com/llvm/llvm-project/pull/102839



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications


More information about the All-commits mailing list