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

[AArch64InstPrinter] Change printAddSubImm to comment imm value when … #3153

Conversation

jasonmolenda
Copy link

[AArch64InstPrinter] Change printAddSubImm to comment imm value when shifted

Add a comment when there is a shifted value,
add x9, x0, #291, lsl #12 ; =1191936
but not when the immediate value is unshifted,
subs x9, x0, #256 ; =256
when the comment adds nothing additional to the reader.

Differential Revision: https://reviews.llvm.org/D107196

(cherry picked from commit 0d8cd4e)

…shifted

Add a comment when there is a shifted value,
    add x9, x0, #291, lsl #12 ; =1191936
but not when the immediate value is unshifted,
    subs x9, x0, #256 ; =256
when the comment adds nothing additional to the reader.

Differential Revision: https://reviews.llvm.org/D107196

(cherry picked from commit 0d8cd4e)
@jasonmolenda
Copy link
Author

@TNorthover I'd like to cherry-pick this aarch64 instprinter tweak into the rebranch.

@jasonmolenda
Copy link
Author

@swift-ci please test

@TNorthover
Copy link

Looks like a good idea to me.

@jasonmolenda jasonmolenda merged commit e2c92f0 into apple/stable/20210628 Aug 10, 2021
@jasonmolenda jasonmolenda deleted the aarch64-instprinter-dont-symbolicate-simple-imms branch August 10, 2021 08:55
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.

2 participants