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
Revert "Replace intrinsics::abort with process::abort" #16899
Conversation
This reverts commit a239419.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
r=me, with a comment on that.
@bors-servo: r=emilio p=1 |
📌 Commit 3fef253 has been approved by |
Revert "Replace intrinsics::abort with process::abort" This reverts #16680. Calling `process::abort` from the crash handler causes the crash handler to be invoked again recursively. --- - [X] `./mach build -d` does not report any errors - [X] `./mach test-tidy` does not report any errors - [X] These changes fix #16898 <!-- Reviewable:start --> --- This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/16899) <!-- Reviewable:end -->
☀️ Test successful - android, arm32, arm64, linux-dev, linux-rel-css, linux-rel-wpt, mac-dev-unit, mac-rel-css, mac-rel-wpt1, mac-rel-wpt2, windows-msvc-dev |
This reverts #16680. Calling
process::abort
from the crash handler causes the crash handler to be invoked again recursively../mach build -d
does not report any errors./mach test-tidy
does not report any errorsThis change is