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

Add initial support -fcoverage-mapping support #13072

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

sbc100
Copy link
Collaborator

@sbc100 sbc100 commented Dec 18, 2020

This is enought make it work up until llvm-cov tries to read the
named data sections in the binary and can't find them. For this
final part to work we probably need to switch the object format to
using multiple code and data sections:
WebAssembly/tool-conventions#138

Not sure if its worth submitting this part in isolation without
a fully working solution?

See #13046

@sbc100 sbc100 mentioned this pull request Dec 18, 2020
This is enought make it work up until llvm-cov tries to read the
named data sections in the binary and can't find them.  For this
final part to work we probably need to switch the object format to
using multiple code and data sections:
WebAssembly/tool-conventions#138

Not sure if its worth submitting this part in isolation without
a fully working solution?

See #13046
Base automatically changed from master to main March 8, 2021 23:49
@stale
Copy link

stale bot commented Apr 16, 2022

This issue has been automatically marked as stale because there has been no activity in the past year. It will be closed automatically if no further activity occurs in the next 30 days. Feel free to re-open at any time if this issue is still relevant.

@stale stale bot added the wontfix label Apr 16, 2022
@AnnieXia
Copy link

emcc -fprofile-instr-generate -fcoverage-mapping demo.cpp

wasm-ld: error: /var/folders/61/x35v93n50_5g1dfjrtprr2dw0000gn/T/emscripten_temp_xjrzk63_/demo_0.o: undefined symbol: __llvm_profile_runtime

need help, emcc seems not support -fcoverage-mapping?

@sbc100
Copy link
Collaborator Author

sbc100 commented Oct 17, 2022

emcc -fprofile-instr-generate -fcoverage-mapping demo.cpp

wasm-ld: error: /var/folders/61/x35v93n50_5g1dfjrtprr2dw0000gn/T/emscripten_temp_xjrzk63_/demo_0.o: undefined symbol: __llvm_profile_runtime

need help, emcc seems not support -fcoverage-mapping?

No, emcc doesn't support this yet. This PR would be a start, but more work would be needed.

@stale stale bot removed the wontfix label Oct 17, 2022
@3d4m-volodymyr
Copy link

Any news on this?

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.

3 participants