[all-commits] [llvm/llvm-project] 283755: [win][clang] Do not inject static_assert macro def...

[Mariya Podchishchaeva via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 28375572f6679b659bb2e8a6535b4b2101b4410e
      https://github.com/llvm/llvm-project/commit/28375572f6679b659bb2e8a6535b4b2101b4410e
  Author: Mariya Podchishchaeva <mariya.podchishchaeva at intel.com>
  Date:   2025-07-08 (Tue, 08 Jul 2025)

  Changed paths:
    M clang/docs/ReleaseNotes.rst
    M clang/include/clang/Basic/DiagnosticGroups.td
    M clang/include/clang/Basic/DiagnosticParseKinds.td
    M clang/lib/Lex/PPDirectives.cpp
    M clang/lib/Parse/ParseDeclCXX.cpp
    M clang/test/Parser/static_assert.c
    R clang/test/Preprocessor/static_assert.c
    M clang/test/Sema/static-assert.c

  Log Message:
  -----------
  [win][clang] Do not inject static_assert macro definition  (#147030)

In ms-compatibility mode we inject static_assert macro definition if
assert macro is defined. This is done by
8da090381d567d0ec555840f6b2a651d2997e4b3
for the sake of better diagnosing, in particular to emit a compatibility
warning when static_assert keyword is used without inclusion of
<assert.h>. Unfortunately it doesn't do a good job in c99 mode adding
that macro unexpectedly for the users, so this patch removes macro
injection and the diagnostics.

---------

Co-authored-by: Corentin Jabot <corentinjabot at gmail.com>



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications