rsmith added a comment. In D140250#4081102 <https://reviews.llvm.org/D140250#4081102>, @rsmith wrote:
> Our builtin header `stddef.h` shouldn't be built as a module. It > fundamentally needs to be treated as a textual header, because it consumes > macros defined by the includer. The module map that we provide with our > builtin headers does not cover `stddef.h` for that reason. That said... the ` __has_feature(modules)` checks throughout `stddef.h` look pretty wrong to me. They're probably working around the infelicities of the `-fno-modules-local-submodule-visibility` mode; maybe we can disable them when not in that mode? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D140250/new/ https://reviews.llvm.org/D140250 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
