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

[InstrRef] Preserve debug instr num in aarch64-expand-pseudo LOADgot expansion #10107

Open
wants to merge 1 commit into
base: stable/20240723
Choose a base branch
from

Conversation

rastogishubham
Copy link

The aarch64-expand-pseudo pass expands the LOADgot instruction to an ADRP instruction and a LDRXui instruction. If the LOADgot had a debug-instr-number, the pass doesn't preserve this to the new expansion.

This patch fixes the issue by making sure the debug-instr-number is correctly applied to the LDRXui instruction generated.

(cherry picked from commit b3c51db)

…expansion (llvm#128081)

The aarch64-expand-pseudo pass expands the LOADgot instruction to an
ADRP instruction and a LDRXui instruction. If the LOADgot had a
debug-instr-number, the pass doesn't preserve this to the new expansion.

This patch fixes the issue by making sure the debug-instr-number is
correctly applied to the LDRXui instruction generated.

(cherry picked from commit b3c51db)
@rastogishubham rastogishubham requested a review from a team as a code owner February 25, 2025 06:23
@rastogishubham
Copy link
Author

@swift-ci please test Linux

@rastogishubham
Copy link
Author

@swift-ci please test macOS

@rastogishubham rastogishubham changed the title [InstrRef] Preserve debug instr num in aarch64-expand-pseudo LOADgot expansion (#128081) [InstrRef] Preserve debug instr num in aarch64-expand-pseudo LOADgot expansion Feb 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant