[PATCH] D129540: [lld-macho] Enable EH frame parsing / pruning

[Nico Weber via Phabricator via llvm-commits]

thakis added a comment.

The patch description of D122258 <https://reviews.llvm.org/D122258> doesn't mention any `-femit-dwarf-unwind` flags, and `-femit-dwarf-unwind=no-compact-unwind` to me isn't very self-explanatory. So just to make sure I got it right, from reading through the diff:

- On Intel, if I pass `-femit-dwarf-unwind=no-compact-unwind`, clang will only emit dwarf unwind information for functions where compact unwind information can't express their unwinding behavior (i.e. almost nothing gets dwarf unwind info)
- On arm, that's the default behavior even without that flag (?)

So we should add `-femit-dwarf-unwind=no-compact-unwind` to our builds, yes? Want me to make repro files at the same chromium rev with and without that flag? Is it sufficient to do that on intel, or does the flag _have_ an effect on arm?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D129540/new/

https://reviews.llvm.org/D129540