[PATCH] D135345: [libunwind] Add module maps for libunwind

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 11 21:49:03 PDT 2022


MaskRay added a comment.

I think the default name `module.modulemap` (loaded by `-fimplicit-module-maps`) is more appropriate for the upstream and it helps layering check features in Clang.
The name `libunwind.modulemap` is indeed special and seems to platform specific. I don't know how to assess `compact_unwind_encoding.modulemap` and the `MachO.*` name instead of `libunwind.*`/`unwind.*`.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D135345



More information about the llvm-commits mailing list