-
Notifications
You must be signed in to change notification settings - Fork 3
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
Coverage reports can't be generated on MacOS #3
Comments
I don't have mac to test it, but probably clang command building object file should have parameter |
Sounds good, thanks @bbarwik! I've tried the Error
I'm afraid that the issue might be in the IR generated when a WASM is built using the Rust toolchain on MacOS, it seems like this is where the COMDAT sections and I suspect that if the LLVM IR is emitted when building the WASM on a linux machine with the linux rust toolchain then the above command would succeed. |
@0xOmarA Try switching your entire shell to x86_64 and then running all commands. |
@njelich-hacken I just tried that and it didn't work. I've tried the following:
None of the above worked and I still pretty much run into the same issue. Also, just to make sure that this is not an individual issue I've reproduced this on a second macbook. |
Hi, this is an issue in
radixdlt/radixdlt-scrypto
but I'm opening an issue here as, to my understanding, you provided the implementation of coverage in that repo.I've been trying to get code coverage to work for sometime now and have unfortunately not been able to. The issues I'm running into seem to be specific to MacOS as I don't seem to run into those issues in a Linux VM.
The error that I run into happens when your tool invokes
clang-17 file.ll -c -Wno-override-module -o file.o
or a variant of that, we get the error seen in the "Error" section.Environment
develop
branch.rustc 1.77.0-nightly (25f8d01fd 2024-01-18)
Steps to reproduce
scrypto new-package coverage; cd coverage; scrypto coverage
Error
Expected Behavior
I expected the above commands to generate the coverage report.
Edit 1: Added the rust toolchain version I have installed.
The text was updated successfully, but these errors were encountered: