Skip to content

Conversation

OCHyams
Copy link
Contributor

@OCHyams OCHyams commented Aug 29, 2025

Key Instructions (-gkey-instructions) is now enabled by default when DWARF is
being emitted, the input is plain C/C++, and optimisations are enabled.

Add release note for the change in default behaviour.

Key Instructions (-gkey-instructions) is now enabled by default when DWARF is
being emitted, the input is plain C/C++, and optimisations are enabled.

Add release note for the change in default behaviour.
@llvmbot llvmbot added the clang Clang issues not falling into any other category label Aug 29, 2025
@llvmbot
Copy link
Member

llvmbot commented Aug 29, 2025

@llvm/pr-subscribers-clang

Author: Orlando Cazalet-Hyams (OCHyams)

Changes

Key Instructions (-gkey-instructions) is now enabled by default when DWARF is
being emitted, the input is plain C/C++, and optimisations are enabled.

Add release note for the change in default behaviour.


Full diff: https://github.com/llvm/llvm-project/pull/156005.diff

1 Files Affected:

  • (modified) clang/docs/ReleaseNotes.rst (+1)
diff --git a/clang/docs/ReleaseNotes.rst b/clang/docs/ReleaseNotes.rst
index 06cc31f8c5c97..739fd50179d25 100644
--- a/clang/docs/ReleaseNotes.rst
+++ b/clang/docs/ReleaseNotes.rst
@@ -211,6 +211,7 @@ Deprecated Compiler Flags
 
 Modified Compiler Flags
 -----------------------
+- The `-gkey-instructions` compiler flag is now enabled by default when DWARF is emitted for plain C/C++ and optimizations are enabled. (#GH149509)
 
 Removed Compiler Flags
 -------------------------

@OCHyams OCHyams requested review from jmorse and SLTozer August 29, 2025 10:51
Copy link
Member

@jmorse jmorse left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@OCHyams OCHyams merged commit 68268c7 into llvm:main Sep 4, 2025
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
clang Clang issues not falling into any other category
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants