Fix unit test that was illegally mutating an upvar

Categorize upvars in `Fn` unboxed closures as freely aliasable

This causes borrowck to correctly reject mutation or mutable borrows
of upvars in `Fn` unboxed closures since the closure environment is
aliasable.

This also tracks the responsible closure in the aliasability
information returned and uses it to give a helpful diagnostic.

Closes issue #17780

Track kind of closure in upvar categorization

Keep track of the kind of closure responsible for an upvar

auto merge of #17769 : dotdash/rust/snapshot, r=eddyb

Register new snapshots

auto merge of #17755 : alexcrichton/rust/unblock-snapshot, r=eddyb

Hopefully unblocks a snapshot

auto merge of #17754 : O-I/rust/update-guide, r=steveklabnik

Hi,

These are a few small edits to the Guide that I made while reading online. Really well done and approachable.

I have a few questions below, but I don't know if this is the proper place to ask them, so feel free to ignore the below.

1. Trailing commas seem to be a convention in Rust and are used quite a bit throughout the Guide, but are never explicitly mentioned. Maybe adding a short mention about them when they first appear in the Structs section might be helpful to those who are unfamiliar with or don't use them in other languages.

2. In the Iterators section, there is a block of code like this:

```rust
let mut range = range(0i, 10i);

loop {
    match range.next() {
        Some(x) => {
            println!("{}", x);
        }  // no comma needed?
        None => { break }
    }
}
```

My inclination would be to put a comma where the comment is to separate the two arms to get this to compile, but it runs fine either way. Is there a convention on commas for scenarios like this where each arm is enclosed in `{}`?

All the best,
O-I

auto merge of #17752 : SimonSapin/rust/patch-6, r=thestinger

`v.len()` counts code units, not UTF-16 bytes. The lower bound is one UTF-8 byte per code unit, not per two code units.

I believe this is correct, but it’s late. Someone please double check.

auto merge of #17731 : bkoropoff/rust/unboxed-by-ref, r=pcwalton

This began as an attempt to fix an ICE in borrowck (issue #17655), but the rabbit hole went pretty deep.  I ended up plumbing support for capture-by-reference unboxed closures all the way into trans.

Closes issue #17655.

test: Fix a test that fails when cross compiling

Hopefully unblocks a snapshot

auto merge of #17743 : steveklabnik/rust/receiver_fix, r=alexcrichton

Adds comma

Oddly (to me), this code runs fine without the comma separating the `Some` and `None` arms of the `match` construct. It seems like Rust doesn't require you to separate arms with commas if all the expressions are enclosed in braces.

Fix preallocation amount in String::from_utf16

`v.len()` counts code units, not UTF-16 bytes. The lower bound is one UTF-8 byte per code unit, not per two code units.

Fixes wording

Removes extra .

Adds missing cd

auto merge of #17742 : alexcrichton/rust/rollup, r=alexcrichton

Trying to get a couple of these into the next snapshot.

Test fixes from the rollup

Use similar syntax in all arms

See issue #17672. This was started in commit ee1cbb9c71bfab8500dfabedb35ba63dd1e5b7ff, but there were a few more lines to update.

Fixes output

Adds trailing comma

Fixes spacing

Update signature of try_recv()

rollup merge of #17739 : eddyb/fix-process-test

rollup merge of #17741 : alexcrichton/travis-again

rollup merge of #17730 : alexcrichton/snapshot

rollup merge of #17729 : alexcrichton/issue-17718-start

rollup merge of #17387 : sneves/master

rollup merge of #17215 : P1start/lints

travis: Fix for real this time

I ended up botching the merge when making the rollup, and the fix was to just
not configure LLVM all via --llvm-root with a nonexistent path.

Fix a race condition between remove_from_env and other io::process tests.

tests: remove old compile-fail test asserting the removal of `const`.

Add some more test coverage of by-ref unboxed closures

Move the lint for the stability lints to the method name only

Closes #17337.

Change rustc pretty-printing to print [T, ..n] instead of [T, .. n]

Report trait/impl sig inconsistency before method/body inconsistency

Closes #15657.

Correct error message for invalid `ref`/`mut` bindings

Closes #15914.

Improve the `non_snake_case` lint to give better suggestions

Update the `unused` lint group to include more lints

Set the `non_uppercase_statics` lint to warn by default

auto merge of #16995 : kmcallister/rust/plugin-tutorial, r=alexcrichton

@steveklabnik, are you interested in looking this over?

auto merge of #17725 : alexcrichton/rust/rollup, r=alexcrichton

Rehabilitate an unboxed closure test

This test works as a regression test for issue #17655.  It also
exercises mutation of by-ref upvars.

Correctly trans capture-by-ref unboxed closures

Store references to the freevars instead of copies when constructing
the environment and insert an additional load when reading them from
the environment.

Return correct types for capture-by-ref unboxed closure upvars

Treat upvars of capture-by-reference unboxed closures as references
with appropriate regions and mutability.

Fix categorization of upvars of capture-by-reference unboxed closures

In particular, this causes mutation of an upvar to correctly mark
it as mutable during adjustment.  This makes borrowck correctly
flag conflicting borrows, etc.

We still seem to generate incorrect code in trans which copies the upvar
by value into the closure.  This remains to be fixed.

Fix missing entries in upvar borrows map for capture-by-ref unboxed closures

This prevents a later ICE in borrowck.

Closes issue #17655

Merge branch 'travis' into rollup

Conflicts:
.travis.yml

syntax: Enable parsing of `const` globals

This rewrites them to the current `ItemStatic` production of the compiler, but I
want to get this into a snapshot. It will be illegal to use a `static` in a
pattern of a `match` statement, so all those current uses will need to be
rewritten to `const` once it's implemented. This requires that the stage0
snapshot is able to parse `const`.

cc #17718

test: Don't depend on /bin/bash

Our FreeBSD bots apparently don't have bash installed, and it's ok to run with
sh anyway!

Unblocks a snapshot

auto merge of #17715 : aturon/rust/revert-slice-ops-libs, r=alexcrichton

This PR reverts https://github.com/rust-lang/rust/pull/17620, which caused a significant regression for slices.

As discussed with @alexcrichton, all of the public-facing changes of the earlier PR need to be rolled back, and it's not clear that we should move the libraries over to this new notation yet anyway (especially given its feature-gated status).

Closes #17710

Test fixes from the rollup

travis: Stop building and testing rust

Instead, only run `make tidy`. The tidy script can run quite quickly, and it's
super annoying to run tests for 50 minutes only to have bors fail with a
"trailing whitespace" error.

rollup merge of #17666 : eddyb/take-garbage-out

Conflicts:
src/libcollections/lib.rs
src/libcore/lib.rs
src/librustdoc/lib.rs
src/librustrt/lib.rs
src/libserialize/lib.rs
src/libstd/lib.rs
src/test/run-pass/issue-8898.rs

rollup merge of #17723 : alexcrichton/travis

rollup merge of #17722 : jakub-/issue-17169

rollup merge of #17721 : jakub-/resolved-issues

rollup merge of #17720 : ben0x539/shifts

rollup merge of #17717 : steveklabnik/gh17190

rollup merge of #17708 : bnoordhuis/fix-ctags-makefile-target

rollup merge of #17706 : thestinger/supp

rollup merge of #17705 : thestinger/realloc

rollup merge of #17702 : nick29581/enable-dots-test

rollup merge of #17698 : jistr/guide_export

rollup merge of #17695 : steveklabnik/various_docs

rollup merge of #17686 : lucidd/fix

rollup merge of #17682 : nodakai/librustc-handy-version

rollup merge of #17646 : bkoropoff/cast-ice

rollup merge of #16993 : dschatzberg/items-bounds

rollup merge of #17719 : alexcrichton/diagnose

rollup merge of #17715 : aturon/revert-slice-ops-libs

travis: Stop building and testing rust

Instead, only run `make tidy`. The tidy script can run quite quickly, and it's
super annoying to run tests for 50 minutes only to have bors fail with a
"trailing whitespace" error.

Add tests for a few resolved issues

std: Help diagnose a flaky test

This test has recently been failing on the bots, and I'm not entirely sure why.
I haven't been able to reproduce locally or on the bots, so I'm adding some
messages to help diagnose the problem hopefully.

Fix cross-crate tuple structs in statics

Fixes #17169.
Fixes #17649.

native: fix passing errno to parent after fork

The bitshifts were wrong in that they invoked undefined behavior and
only passed the lower byte of the presumed-to-be-32bit errno value.
Apparently all actually possible values for errno happen to be easily
under 256, so this didn't cause any actual problems.

This commit fixes the bitshifts, but doesn't generalize to errno types
that aren't 32bit.

alloc: fix reallocate_inplace implementation

The returned size is the new real size of the allocation.

extra comment about macros

Fixes #17190

I am bad at math

Revert "Use slice syntax instead of slice_to, etc."

This reverts commit 40b9f5ded50ac4ce8c9323921ec556ad611af6b7.

Revert "Remove the `_` suffix from slice methods."

This reverts commit df2f1fa7680a86ba228f004e7de731e91a1df1fe.

Revert "Put slicing syntax behind a feature gate."

This reverts commit 95cfc35607ccf5f02f02de56a35a9ef50fa23a82.

Revert "Review and rebasing changes"

This reverts commit 6e0611a48707a1f5d90aee32a02b2b15957ef25b.

Disable the Roman numerals test on Android due to cross-compile issues

Add fixes for new lifetime bounds

auto merge of #17590 : bjadamson/rust/rustc-improvements, r=alexcrichton

Removes an unnecessary allocation when passing the command line arguments to the librustc driver.

Add tests for MoveItems

Use RawPtr::offset when size_of::<T>() > 0

Add lifetime bounds on Items and MutItems.

This also requires a fix for Vec's MoveItems. This resolves issue #16941

tests: remove uses of Gc.

syntax: remove ObsoleteManaged{Type,Expr}.

syntax: mark the managed_boxes feature as Removed.

docs: remove mentions of Gc.

rustdoc: remove handling of Gc.

rustc: remove support for Gc.

syntax: ast: remove TyBox and UnBox.

syntax: remove unused imports of Gc and GC.

debug: remove Gc support from Repr.

serialize: remove proxy impls for Gc<T>.

std: remove gc module.