[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