Crates using placement syntax behind a cfg broken

[emilyalbini]

Some crates were using <- behind a feature flag, so they worked on stable even if the placement syntax was nightly-only. After removing it though they fail to compile even on stable with a syntax error, because <- is not recognized anymore.

• mbox regressed from stable to beta (build log) cc @kennytm
• rayon-hash regressed from stable to beta (build log) cc @cuviper

[kennytm]

It is already fixed in mbox 0.5.0 and 0.4.4.

[nikomatsakis]

Oh, that's annoying. We should patch rayon-hash.

[petrochenkov]

This is not the first time this is happening, removal of impl Trait for .. {} caused same consequences (see #46480).
We should probably continue parsing <- and report the error later in AST validation.

[nikomatsakis]

Discussed in the @rust-lang/compiler meeting. I think we reached this consensus:

• in this case, yes, we should do as @petrochenkov suggests: keep parsing, error in AST validation
• if in the future, if it happens that we want to repurpose the syntax, we deal with it then: best of course is to use an edition for it, but maybe by then there no longer exist crates using it
• going forward, it would be ideal if we could do some kind of "pre-cfg / expansion" feature check to try and avoid this in the future; that may or may not be feasible though, and we couldn't do it retroactively for all existing things anyway

[nikomatsakis]

Some discussion of the final at or around here in the IRC logs

https://botbot.me/mozilla/rustc/2018-05-17/?msg=100165163&page=3

[nikomatsakis]

triage: P-high

[nikomatsakis]

Would anyone have time to whip up a change to the parser?

cc @rust-lang/compiler @aidanhs

[nikomatsakis]

For reference, the PR that reverted the syntax was #48333