about summary refs log tree commit diff
path: root/src/rustllvm/RustWrapper.cpp
diff options
context:
space:
mode:
authorbors <bors@rust-lang.org>2018-04-03 11:50:11 +0000
committerbors <bors@rust-lang.org>2018-04-03 11:50:11 +0000
commit637ac17c5292ce723430de0e87b92271989e1436 (patch)
tree4e1ed24e03373c2473660a352c1aaa0f823511e2 /src/rustllvm/RustWrapper.cpp
parentb12af86a776364f01f65e3bd551e2641b4fac4ec (diff)
parent347cf21290c80c528363cc33d9cade6cd1e77888 (diff)
downloadrust-637ac17c5292ce723430de0e87b92271989e1436.tar.gz
rust-637ac17c5292ce723430de0e87b92271989e1436.zip
Auto merge of #49447 - pnkfelix:remove-cfg-const-pat-hack-47295, r=nikomatsakis
Remove adjacent all-const match arm hack.

An old fix for moves-in-guards had a hack for adjacent all-const match arms.

The hack was explained in a comment, which you can see here:
https://github.com/rust-lang/rust/pull/22580/files#diff-402a0fa4b3c6755c5650027c6d4cf1efR497

But hack was incomplete (and thus unsound), as pointed out here:
https://github.com/rust-lang/rust/issues/47295#issuecomment-357108458

Plus, it is likely to be at least tricky to reimplement this hack in
the new NLL borrowck.

So rather than try to preserve the hack, we want to try to just remove
it outright. (At least to see the results of a crater run.)

[breaking-change]

This is a breaking-change, but our hope is that no one is actually
relying on such an extreme special case. (We hypothesize the hack was
originally added to accommodate a file in our own test suite, not code
in the wild.)
Diffstat (limited to 'src/rustllvm/RustWrapper.cpp')
0 files changed, 0 insertions, 0 deletions