Skip to content

Commit 0212ff5

Browse files
author
Doug Wyatt
committed
Revert formatting change and release note move. Fix test broken by this change.
1 parent 12dc263 commit 0212ff5

File tree

3 files changed

+6
-6
lines changed

3 files changed

+6
-6
lines changed

clang/docs/ReleaseNotes.rst

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -350,8 +350,8 @@ Improvements to Clang's diagnostics
350350
Moved the warning for a missing (though implied) attribute on a redeclaration into this group.
351351
Added a new warning in this group for the case where the attribute is missing/implicit on
352352
an override of a virtual method.
353-
- Fixed several false positives and false negatives in function effect (`nonblocking`) analysis. (#GH166078) (#GH166101) (#GH166110)
354-
- Remove ``-Wperf-constraint-implies-noexcept`` from ``-Wall``.
353+
- Remove ``-Wperf-constraint-implies-noexcept`` from ``-Wall``. This warning is somewhat pedantic and
354+
attempts to resolve it, by adding ``noexcept``, can create new ways for programs to crash. (#GH167540)
355355
- Implemented diagnostics when retrieving the tuple size for types where its specialization of `std::tuple_size`
356356
produces an invalid size (either negative or greater than the implementation limit). (#GH159563)
357357
- Fixed fix-it hint for fold expressions. Clang now correctly places the suggested right
@@ -484,6 +484,7 @@ Bug Fixes to Attribute Support
484484
- Fix a crash when the function name is empty in the `swift_name` attribute. (#GH157075)
485485
- Fixes crashes or missing diagnostics with the `device_kernel` attribute. (#GH161905)
486486
- Fix handling of parameter indexes when an attribute is applied to a C++23 explicit object member function.
487+
- Fixed several false positives and false negatives in function effect (`nonblocking`) analysis. (#GH166078) (#GH166101) (#GH166110)
487488

488489
Bug Fixes to C++ Support
489490
^^^^^^^^^^^^^^^^^^^^^^^^

clang/include/clang/Basic/DiagnosticGroups.td

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1312,9 +1312,9 @@ def Consumed : DiagGroup<"consumed">;
13121312
// Note that putting warnings in -Wall will not disable them by default. If a
13131313
// warning should be active _only_ when -Wall is passed in, mark it as
13141314
// DefaultIgnore in addition to putting it here.
1315-
def All
1316-
: DiagGroup<"all", [Most, Parentheses, Switch, SwitchBool,
1317-
MisleadingIndentation, PackedNonPod, VLACxxExtension]>;
1315+
def All : DiagGroup<"all", [Most, Parentheses, Switch, SwitchBool,
1316+
MisleadingIndentation, PackedNonPod,
1317+
VLACxxExtension]>;
13181318

13191319
// Warnings that should be in clang-cl /w4.
13201320
def : DiagGroup<"CL4", [All, Extra]>;

clang/test/Misc/warning-wall.c

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -109,6 +109,5 @@ CHECK-NEXT: -Wmisleading-indentation
109109
CHECK-NEXT: -Wpacked-non-pod
110110
CHECK-NEXT: -Wvla-cxx-extension
111111
CHECK-NEXT: -Wvla-extension-static-assert
112-
CHECK-NEXT: -Wperf-constraint-implies-noexcept
113112

114113
CHECK-NOT:-W

0 commit comments

Comments
 (0)