[all-commits] [llvm/llvm-project] ce4ac9: [libc++] Remove explicit mentions of __need_FOO ma...
Louis Dionne via All-commits
all-commits at lists.llvm.org
Tue Dec 17 06:52:56 PST 2024
Branch: refs/heads/main
Home: https://github.com/llvm/llvm-project
Commit: ce4ac994529eb33a67b0f450fd6fe847918247e1
https://github.com/llvm/llvm-project/commit/ce4ac994529eb33a67b0f450fd6fe847918247e1
Author: Louis Dionne <ldionne.2 at gmail.com>
Date: 2024-12-17 (Tue, 17 Dec 2024)
Changed paths:
M libcxx/include/module.modulemap
M libcxx/include/stdio.h
M libcxx/include/stdlib.h
M libcxx/include/wchar.h
M libcxx/test/libcxx/clang_modules_include.gen.py
Log Message:
-----------
[libc++] Remove explicit mentions of __need_FOO macros (#119025)
This change has a long history. It was first attempted naively in
https://reviews.llvm.org/D131425, which didn't work because we broke the
ability for code to include e.g. <stdio.h> multiple times and get
different definitions based on the pre-defined macros.
However, in #86843 we managed to simplify <stddef.h> by including the
underlying system header outside of any include guards, which worked.
This patch applies the same simplification we did to <stddef.h> to the
other headers that currently mention __need_FOO macros explicitly.
To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications
More information about the All-commits
mailing list