Skip to content

Commit

Permalink
[MachO] Remove stale comments
Browse files Browse the repository at this point in the history
https://reviews.llvm.org/D93267 implemented handling more than 127
compact unwind encodings, and https://reviews.llvm.org/D123435 and
https://reviews.llvm.org/D124561 implemented stripping redundant
__eh_frame entries.
  • Loading branch information
smeenai committed Aug 28, 2022
1 parent a284f9f commit f0c0094
Showing 1 changed file with 0 additions and 8 deletions.
8 changes: 0 additions & 8 deletions lld/MachO/UnwindInfoSection.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -82,18 +82,10 @@ using namespace lld::macho;
// advantage, achieving a 3-order-of-magnitude reduction in the
// number of entries.
//
// * The __TEXT,__unwind_info format can accommodate up to 127 unique
// encodings for the space-efficient compressed format. In practice,
// fewer than a dozen unique encodings are used by C++ programs of
// all sizes. Therefore, we don't even bother implementing the regular
// non-compressed format. Time will tell if anyone in the field ever
// overflows the 127-encodings limit.
//
// Refer to the definition of unwind_info_section_header in
// compact_unwind_encoding.h for an overview of the format we are encoding
// here.

// TODO(gkm): prune __eh_frame entries superseded by __unwind_info, PR50410
// TODO(gkm): how do we align the 2nd-level pages?

// The offsets of various fields in the on-disk representation of each compact
Expand Down

0 comments on commit f0c0094

Please sign in to comment.