Assert error in resolveAArch64Relocation about bolt

[yota9]

Hello! What BOLT version do you use? Using the latest bolt with this commit (llvm/llvm-project@4c14519) should fix the problem

[HShan886]

@yota9 thank you for replying.
I am using the latest bolt, and this commit has been merged into the latest bolt. The commit doesn't work.

[yota9]

@Haishan312 May I see the binary or at least function objdump + readelf section/segments output? Probably the binary would be preferable way, since I need to see exactly what is going on there

[HShan886]

@yota9 You can download the binary from this repo git clone https://github.com/Haishan312/bolt-fixbugs.git. Because of github issue limit the size of attach file.

[yota9]

@Haishan312 Yes, I was able to download it, thanks. Also I need your bolt options (and might be fdata file) to reproduce the issue

[HShan886]

@yota9 bolt options is llvm-bolt ${binary_name} -o ${binary_name}.bolt -data=perf.fdata -reorder-blocks=cache+ -reorder-functions=hfsort -split-functions=2 -split-all-cold -split-eh -dyno-stats --update-debug-sections -v=2

and the fdata file
502.fdata.zip

[yota9]

@Haishan312 Thanks, I was able to repro the issue, it is not related to plt, but I will try to resolve it.

[HShan886]

@yota9 that's great, thank you.
I have another case which is related to plt. I am recovering a small binary

[yota9]

Hello @Haishan312 ! Sorry for long reply. I've checked out your binary, the problem is known to me and I've raised it earlier with goland support. Nevertheless I've found the way to fix it easily with this commit llvm/llvm-project@fd96049 , please try to rebase the bolt on the latest commits and check if there are any other problems. Thank you!

[HShan886]

@yota9 Thank you very much. This commit resolves my problem.

[yota9]

@Haishan312 Great!
Please close the issue if everything is solved :)

[HShan886]

@yota9 Your patch would generate an illegal instruction on aarch64 platform. when I use llvm-objdump tools to dump assembly. I get the an unknown instruction, such as:
ae3254: 60 d9 72 b8 ldr w0, [x11, w18, sxtw #2]
ae3258: 00 b2 fc 7f unknown
ae325c: e0 03 11 aa mov x0, x17

And you can download original attach files.
illegel_instr.tar.gz

[yota9]

Hello @Haishan312 ! I don't think the patch has anything to do with that. For some reason tbz instruction was not relaxed, I would check the reason soon. Thank you for reporting!

[yota9]

@Haishan312 The problem was found in another part of LLVM https://reviews.llvm.org/D128740 . It may take some time before approve would be received, so fill free to apply this patch manually :)
UPD The patch has landed.

[aaupov]

Assuming the issue is resolved with rGb27d6ffe4e4a.