Skip to content

ci: Reenable disabled sanitizer options in sanitizers-gcc#818

Merged
axic merged 1 commit intomasterfrom
ci-sanitizers
May 23, 2022
Merged

ci: Reenable disabled sanitizer options in sanitizers-gcc#818
axic merged 1 commit intomasterfrom
ci-sanitizers

Conversation

@axic
Copy link
Copy Markdown
Member

@axic axic commented May 23, 2022

These bugs have been fixed.

@axic axic requested review from chfast and gumb0 May 23, 2022 09:39
@codecov
Copy link
Copy Markdown

codecov Bot commented May 23, 2022

Codecov Report

Merging #818 (3464161) into master (ee99be7) will decrease coverage by 0.25%.
The diff coverage is n/a.

❗ Current head 3464161 differs from pull request most recent head f757ac0. Consider uploading reports for the commit f757ac0 to get more accurate results

@@            Coverage Diff             @@
##           master     #818      +/-   ##
==========================================
- Coverage   99.35%   99.10%   -0.26%     
==========================================
  Files          86       86              
  Lines       13174    13017     -157     
==========================================
- Hits        13089    12900     -189     
- Misses         85      117      +32     
Flag Coverage Δ
rust 99.60% <ø> (ø)
spectests 89.98% <ø> (ø)
unittests 98.94% <ø> (-0.28%) ⬇️
unittests-32 99.03% <ø> (-0.28%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
tools/wasi/wasi.cpp 60.18% <0.00%> (-32.41%) ⬇️
test/unittests_wasi/wasi_test.cpp
test/unittests/wasi_test.cpp 100.00% <0.00%> (ø)

@axic axic force-pushed the ci-sanitizers branch from 3464161 to 42b0ffc Compare May 23, 2022 10:27
Copy link
Copy Markdown
Collaborator

@chfast chfast left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The bad part is that the compiler is auto-upgraded?

@axic
Copy link
Copy Markdown
Member Author

axic commented May 23, 2022

The bad part is that the compiler is auto-upgraded?

We have that problem generally?

@axic axic force-pushed the ci-sanitizers branch from 42b0ffc to f757ac0 Compare May 23, 2022 14:35
@axic axic merged commit 9ae449c into master May 23, 2022
@axic axic deleted the ci-sanitizers branch May 23, 2022 15:47
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