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

Rollup of 3 pull requests #63200

Closed
wants to merge 25 commits into from
Closed

Conversation

Centril
Copy link
Contributor

@Centril Centril commented Aug 1, 2019

Successful merges:

Failed merges:

r? @ghost

varkor and others added 25 commits July 31, 2019 01:51
There is no difference in the code/arguments, so go with the shorter
name throughout the code.
MetaItem.span was always equivalent
Always the same as the passed MetaItem
…en dispatching further; call ptr_op machine hook only for pointer types
Co-Authored-By: Mazdak Farrokhzad <twingoow@gmail.com>
…r=oli-obk

Miri: dispatch first on the type

Based on the fact that Miri now always has intptrcast available, we can change binops and casts to first check the type of the source operand and then decide based on that what to do, instead of considering the value (pointer vs bits) first.
…chenkov

Cleanup syntax::attr

Mostly removing needless arguments to constructors

r? @petrochenkov
…cramertj

Remove redundant method with const variable resolution

I've also removed a `bug!()` in const value relation code and replaced it with a `FIXME`. Now `ConstValue::Slice` and `ConstValue::ByRef` will simply fail to unify rather than ICEing, which seems more user-friendly for testers.
@Centril
Copy link
Contributor Author

Centril commented Aug 1, 2019

@bors r+ p=3 rollup=never

@bors
Copy link
Contributor

bors commented Aug 1, 2019

📌 Commit 3faef22 has been approved by Centril

@bors bors added the S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. label Aug 1, 2019
@bors
Copy link
Contributor

bors commented Aug 2, 2019

⌛ Testing commit 3faef22 with merge 702e3d08069f813159813fe3ac87974e1f4ed7c2...

@bors
Copy link
Contributor

bors commented Aug 2, 2019

💥 Test timed out

@bors bors added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. and removed S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. labels Aug 2, 2019
@Centril Centril closed this Aug 2, 2019
@Centril Centril deleted the rollup-tahv30c branch August 2, 2019 07:45
@Centril Centril added the rollup A PR which is a rollup label Oct 24, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
rollup A PR which is a rollup S-waiting-on-review Status: Awaiting review from the assignee but also interested parties.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants