[all-commits] [llvm/llvm-project] 898c67: [C2x] Remove the ATOMIC_VAR_INIT macro from stdato...

Aaron Ballman via All-commits all-commits at lists.llvm.org
Fri Feb 24 10:54:36 PST 2023


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 898c673e0835a2df395dd2476ac8ee8972d6380b
      https://github.com/llvm/llvm-project/commit/898c673e0835a2df395dd2476ac8ee8972d6380b
  Author: Aaron Ballman <aaron at aaronballman.com>
  Date:   2023-02-24 (Fri, 24 Feb 2023)

  Changed paths:
    M clang/docs/ReleaseNotes.rst
    M clang/lib/Headers/stdatomic.h
    A clang/test/C/C2x/n2886.c
    M clang/www/c_status.html

  Log Message:
  -----------
  [C2x] Remove the ATOMIC_VAR_INIT macro from stdatomic.h

This implements WG14 N2886 (https://www.open-std.org/jtc1/sc22/wg14/www/docs/n2886.htm)
which removed the macro entirely. (NB the macro was deprecated in C17.)
As the paper is not particularly clear on what alternative was picked,
here are my notes from the May 2022 meeting:

Does WG14 wish to adopt variant 1, change 3.2, 3.3, and 3.4 from N2886
into C23?
14/2/2 (consensus)
Does WG14 want to exchange Variant 1 with Variant 2 in N2886 in C23?
9/3/6 (consensus)

(There was no sentiment in the room for either Variant 3 or Variant 4
so those were not voted on.)

Does WG14 want to integrate change 3.5 in N2886 into C23?
8/1/9 (consensus)
Does WG14 want to integrate change 3.6 in N2886 into C23?
2/5/9 (no consensus)

Any code that is broken by the removal can remove the use of
ATOMIC_VAR_INIT and use regular initialization instead.

Differential Revision: https://reviews.llvm.org/D144196




More information about the All-commits mailing list