llvmorg-github-actions[bot] wrote:
<!--LLVM PR SUMMARY COMMENT--> @llvm/pr-subscribers-clang Author: sipher (sipher-01) <details> <summary>Changes</summary> Fixes #<!-- -->204213 `-Wlifetime-safety-dangling-global-moved` was defined but not included in any parent diagnostic group, meaning `-Wlifetime-safety-all` and `-Wno-lifetime-safety-all` had no effect on it. `LifetimeSafetyDanglingGlobal` (the non-moved variant) was already correctly placed under `LifetimeSafetyPermissive`. Following the same pattern as the other `-moved` sibling warnings (`DanglingFieldMoved`, `ReturnStackAddrMoved`, `UseAfterScopeMoved`), this patch adds `LifetimeSafetyDanglingGlobalMoved` to `LifetimeSafetyStrict`, which is part of the `lifetime-safety-all` hierarchy. --- Full diff: https://github.com/llvm/llvm-project/pull/204891.diff 1 Files Affected: - (modified) clang/include/clang/Basic/DiagnosticGroups.td (+1) ``````````diff diff --git a/clang/include/clang/Basic/DiagnosticGroups.td b/clang/include/clang/Basic/DiagnosticGroups.td index 244cd3630bb11..5cdebd35ba05d 100644 --- a/clang/include/clang/Basic/DiagnosticGroups.td +++ b/clang/include/clang/Basic/DiagnosticGroups.td @@ -663,6 +663,7 @@ def LifetimeSafetyStrict : DiagGroup<"lifetime-safety-strict", LifetimeSafetyUseAfterScopeMoved, LifetimeSafetyReturnStackAddrMoved, LifetimeSafetyDanglingFieldMoved, + LifetimeSafetyDanglingGlobalMoved, LifetimeSafetyInvalidation]>; def LifetimeSafety : DiagGroup<"lifetime-safety", `````````` </details> https://github.com/llvm/llvm-project/pull/204891 _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
