[libcxx-commits] [PATCH] D105078: [libcxx][modularisation] properly modularises advance, next, and prev
Arthur O'Dwyer via Phabricator via libcxx-commits
libcxx-commits at lists.llvm.org
Wed Jul 21 15:08:38 PDT 2021
Quuxplusone requested changes to this revision.
Quuxplusone added inline comments.
This revision now requires changes to proceed.
================
Comment at: libcxx/include/module.modulemap:481
module access { private header "__iterator/access.h" }
- module advance { private header "__iterator/advance.h" }
+ module advance {
+ private header "__iterator/advance.h"
----------------
zoecarver wrote:
> I don't like the formatting here. But whatever.
There should be no need to `export __function_like` here. Then it'll all fit on one line, consistent with all the other lines, so there won't be a formatting problem anymore.
================
Comment at: libcxx/include/module.modulemap:786
module __errc { private header "__errc" export * }
module __function_like { private header "__function_like.h" export * }
module __hash_table { header "__hash_table" export * }
----------------
Probably this `private` is the original problem. What happens if you remove it?
(Compare to `__hash_table`, `__nullptr`, etc.)
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D105078/new/
https://reviews.llvm.org/D105078
More information about the libcxx-commits
mailing list