Skip to content

Merge r347261 and maybe r347262 into the 7.0 branch (Clang coverage crash fixes) #39087

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

Closed
orivej opened this issue Nov 21, 2018 · 5 comments
Closed
Assignees
Labels
bugzilla Issues migrated from bugzilla

Comments

@orivej
Copy link
Contributor

orivej commented Nov 21, 2018

Bugzilla Link 39740
Resolution FIXED
Resolved on Nov 28, 2018 12:04
Version 7.0
OS Linux
Blocks #38454
CC @tstellar,@vedantk
Fixed by commit(s) r347261 r347262 r347797 r347798

Extended Description

https://reviews.llvm.org/rL347261 is needed because it fixes a regression reported in #38335

https://reviews.llvm.org/rL347262 may be merged at the discretion of the code owners and release managers because it fixes the last known crash in Clang coverage.

@orivej
Copy link
Contributor Author

orivej commented Nov 21, 2018

assigned to @tstellar

@tstellar
Copy link
Collaborator

Richard, what do you think about these? I can try to find another review for the code coverage fix if you aren't familiar with that code.

@zygoloid
Copy link
Mannequin

zygoloid mannequin commented Nov 27, 2018

Both patches are fine for backporting.

@tstellar
Copy link
Collaborator

Merged: r347797 r347798

@orivej
Copy link
Contributor Author

orivej commented Nov 27, 2021

mentioned in issue llvm/llvm-bugzilla-archive#39822

@llvmbot llvmbot transferred this issue from llvm/llvm-bugzilla-archive Dec 10, 2021
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bugzilla Issues migrated from bugzilla
Projects
None yet
Development

No branches or pull requests

2 participants