[all-commits] [llvm/llvm-project] 2cb3d2: [lld/mac] Add some comments and asserts

Nico Weber via All-commits all-commits at lists.llvm.org
Thu Apr 7 12:43:52 PDT 2022


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 2cb3d28b173570c4b34c5ce85bc18713670d07e1
      https://github.com/llvm/llvm-project/commit/2cb3d28b173570c4b34c5ce85bc18713670d07e1
  Author: Nico Weber <thakis at chromium.org>
  Date:   2022-04-07 (Thu, 07 Apr 2022)

  Changed paths:
    M lld/MachO/SymbolTable.cpp
    M lld/MachO/SyntheticSections.cpp

  Log Message:
  -----------
  [lld/mac] Add some comments and asserts

I was wondering if SymtabSection::emitStabs() should check
defined->includeInSymtab. Add asserts and comments explaining why that's not
necessary.

No behavior change.

Differential Revision: https://reviews.llvm.org/D123302




More information about the All-commits mailing list