ndk icon indicating copy to clipboard operation
ndk copied to clipboard

[FR] Add pdbs to the NDK

Open tiagomacarios opened this issue 1 year ago • 6 comments

Description

See #2059 for background. Having the clang pdbs would help us understand what the issue is and possibly suggest fixes/work around issues.

tiagomacarios avatar Aug 22 '24 23:08 tiagomacarios

@pirama-arumuga-nainar, WDYT? Maybe not include them directly in the NDK (are they large?), but make sure they're on ci.android.com so people can get them if they need them?

DanAlbert avatar Aug 23 '24 19:08 DanAlbert

LLVM_ENABLE_PDB should generate this. I'm testing locally to get an idea about the size.

ci.android.com sounds like the correct place to make them available. Note that our current builds have a shorter retention period (internal bug b/332650194) but that'll probably get resolved within the next two quarters.

pirama-arumuga-nainar avatar Aug 23 '24 19:08 pirama-arumuga-nainar

$ du -sh clang.pdb lld.pdb
896M    clang.pdb
565M    lld.pdb

These are several times larger than clang itself. Both of the files compress down to 214M combined. Definitely not something we could distribute.

pirama-arumuga-nainar avatar Aug 23 '24 22:08 pirama-arumuga-nainar

They're rarely needed, so ci.android.com would be fine anyway.

DanAlbert avatar Aug 23 '24 23:08 DanAlbert

@DanAlbert @pirama-arumuga-nainar Was this implemented? If yes, could you document where to find the pdbs? I am not very familiar with the build infrastructure.

tiagomacarios avatar Sep 03 '24 17:09 tiagomacarios

Not yet. We've got a lot of other things on our plate. We'll update the bug when something changes.

DanAlbert avatar Sep 03 '24 19:09 DanAlbert