-
Notifications
You must be signed in to change notification settings - Fork 80
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
All SVE functions cannot be debugged with GDB because of the missing .text declaration in the asm file. #125
Comments
Thanks for reporting @chenxuqiang. Anyone from ARM that can review this issue and issue #124? Also, @chenxuqiang, feel free to submit a PR with these changes. |
This issue relates to gcc (or GNU as)'s default behavior actually. It's not a SVE assembly issue. I explained in #127 (comment) I would suggest to add '.text' declaration explicitly in all .S files where it's needed. |
@chenxuqiang, could you look into the other .S files? |
Thanks @docularxu @pablodelara , I'm going to fix it before April 7th. |
Thanks @chenxuqiang! |
@chenxuqiang We are looking at a release next month, so could you look into this as soon as you can? Thanks! |
I found that all SVE functions cannot be debugged with GDB. This is because the code segment identifier of
.text
is missing in the SVE function code implementation. As a result, GDB cannot find executable assembly code. You can use theobjdump -d
to verify this problem:# objdump -d sm3_mb/aarch64/sm3_mb_sve.o sm3_mb/aarch64/sm3_mb_sve.o: file format elf64-littleaarch64 [no assembler contents of executable sections]
If we add
.text
to the sm3_mb/aarch64/sm3_mb_sve.S:We can see:
and use GDB Debug this function:
So I think we should add
.text
for all SVE functions, including:md5_mb_sve.S
,mh_sha1_block_sve.S
,sm3_mb_sve.S
. and and should be followed by#inlude "xxxxx.S"
.The text was updated successfully, but these errors were encountered: