-
Notifications
You must be signed in to change notification settings - Fork 12.8k
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
[beta] backports #42927
[beta] backports #42927
Conversation
config.mk is now always read when parsing the configuration to prevent this from reoccurring in the future, hopefully.
So ARM had quite a few codegen bugs on LLVM 4.0 which are fixed on LLVM trunk. This backports 5 of them: r297871 - ARM: avoid clobbering register in v6 jump-table expansion. - fixes rust-lang#42248 r294949 - [Thumb-1] TBB generation: spot redefinitions of index r295816 - [ARM] Fix constant islands pass. r300870 - [Thumb-1] Fix corner cases for compressed jump tables r302650 - [IfConversion] Add missing check in IfConversion/canFallThroughTo - unblocks rust-lang#39409
This reverts commit 65d0be3.
r? @arielb1 (rust_highfive has picked a reviewer for you, use r? to override) |
@bors r=alexcrichton p=1 (The other PR was approved already) |
📌 Commit fc87577 has been approved by |
⌛ Testing commit fc87577 with merge 665f8dc28a83bad2c34641d14e637e8fc90cb661... |
☀️ Test successful - status-appveyor, status-travis |
@bors force clean retry |
💡 This pull request was already approved, no need to approve it again. |
📌 Commit fc87577 has been approved by |
Actually, I think I remember now -- we disabled force(?) pushes to the beta branch, which is why this is having problems I think. I'll r- for now, we should do that and then r+ this. @alexcrichton -- can you do that? Also pinged on IRC. @bors r- |
@bors r=alexcrichton p=1 |
📌 Commit fc87577 has been approved by |
💔 Test failed - status-travis |
Hm, that's the sanitizer leak failure again. I'm suspicious, but still think that it's spurious. Let's try one more time. @bors retry |
☀️ Test successful - status-appveyor, status-travis |
[beta] rebase LLVM on top of 4.0.1 This backports fixes several codegen bugs, including #42893. This also un-commits the introduction of #42750 (the StackColoring improvement) to beta. That seemed to have occurred by accident in #42927, but I'm not comfortable with sneaking a subtle codegen change like this to beta. r? @alexcrichton
[beta] Bump prerelease version There are some more backports left to do, but I figure it's best to get this bump in so we can release [this massive set of backports](#42927). r? @Mark-Simulacrum p=1 maybe
Reopening #42841 as a new PR due to bors flakiness.
break
does not affect type inference. Fixes #42618 #42634I just unilaterally accepted all the non-accepted nominations. Everything picked cleanly.