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

test the thumbv7em-none-eabihf target #68

Merged
merged 1 commit into from
Sep 22, 2016
Merged

test the thumbv7em-none-eabihf target #68

merged 1 commit into from
Sep 22, 2016

Conversation

japaric
Copy link
Member

@japaric japaric commented Sep 22, 2016

No description provided.

@japaric
Copy link
Member Author

japaric commented Sep 22, 2016

@homunkulus r+

@homunkulus
Copy link
Contributor

📌 Commit 0404438 has been approved by japaric

@homunkulus
Copy link
Contributor

⌛ Testing commit 0404438 with merge 0404438...

@homunkulus
Copy link
Contributor

💔 Test failed - status-travis

@japaric
Copy link
Member Author

japaric commented Sep 22, 2016

@homunkulus r+

@homunkulus
Copy link
Contributor

💡 This pull request was already approved, no need to approve it again.

  • This pull request previously failed. You should add more commits to fix the bug, or use retry to trigger a build again.
  • There's another pull request that is currently being tested, blocking this pull request: avoid division overflow in idivmod test #69

@homunkulus
Copy link
Contributor

📌 Commit 0404438 has been approved by japaric

@japaric
Copy link
Member Author

japaric commented Sep 22, 2016

hmm, @homunkulus retry

@homunkulus
Copy link
Contributor

⌛ Testing commit 0404438 with merge fc2a502...

japaric pushed a commit that referenced this pull request Sep 22, 2016
test the thumbv7em-none-eabihf target
@japaric japaric merged commit 9e50598 into master Sep 22, 2016
@japaric
Copy link
Member Author

japaric commented Sep 22, 2016

The test passed but I think homu was asleep when that ocurred.

Merging manually.

@japaric japaric deleted the another-thumb branch September 22, 2016 17:24
tgross35 pushed a commit to tgross35/compiler-builtins that referenced this pull request Feb 23, 2025
68: catch panics in tests r=japaric a=japaric

so we can print the inputs that triggered the panic

Co-authored-by: Jorge Aparicio <[email protected]>
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.

2 participants