Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[LangRef] Refine header for "Preserving DI Intrinsics" section #67908

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

vadorovsky
Copy link
Contributor

Prior to this change, the "Preserving DI Intrinsics" section was incorrectly headered, causing the documentation about BPF DI preserving intrinsics to be misplaced under the "Objective-C ARC Runtime Intrinsics" section in the document hierarchy.

This change corrects the header for the "Preserving DI Intrinsics" section, ensuring it is properly placed within the documentation hierarchy as its own distinct section.

Prior to this change, the "Preserving DI Intrinsics" section was incorrectly
headered, causing the documentation about BPF DI preserving intrinsics to be
misplaced under the "Objective-C ARC Runtime Intrinsics" section in the
document hierarchy.

This change corrects the header for the "Preserving DI Intrinsics" section,
ensuring it is properly placed within the documentation hierarchy as its own
distinct section.
@llvmbot llvmbot added the llvm:ir label Oct 1, 2023
@vadorovsky vadorovsky changed the title [LangRef] Refine Header for "Preserving DI Intrinsics" Section [LangRef] Refine header for "Preserving DI Intrinsics" section Oct 1, 2023
@llvmbot
Copy link
Collaborator

llvmbot commented Oct 1, 2023

@llvm/pr-subscribers-llvm-ir

Changes

Prior to this change, the "Preserving DI Intrinsics" section was incorrectly headered, causing the documentation about BPF DI preserving intrinsics to be misplaced under the "Objective-C ARC Runtime Intrinsics" section in the document hierarchy.

This change corrects the header for the "Preserving DI Intrinsics" section, ensuring it is properly placed within the documentation hierarchy as its own distinct section.


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

1 Files Affected:

  • (modified) llvm/docs/LangRef.rst (+2-2)
diff --git a/llvm/docs/LangRef.rst b/llvm/docs/LangRef.rst
index c8f74c19bd6b3cf..4858e87645a0df1 100644
--- a/llvm/docs/LangRef.rst
+++ b/llvm/docs/LangRef.rst
@@ -27440,12 +27440,12 @@ Lowering:
 Lowers to a call to `objc_storeWeak <https://clang.llvm.org/docs/AutomaticReferenceCounting.html#arc-runtime-objc-storeweak>`_.
 
 Preserving Debug Information Intrinsics
-^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
+---------------------------------------
 
 These intrinsics are used to carry certain debuginfo together with
 IR-level operations. For example, it may be desirable to
 know the structure/union name and the original user-level field
-indices. Such information got lost in IR GetElementPtr instruction
+indices. Such information gets lost in IR GetElementPtr instruction
 since the IR types are different from debugInfo types and unions
 are converted to structs in IR.
 

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants