Place::Local(x) is now Place::Base(PlaceBase::Local(x))

[spastorino]

We need to merge this after the beta cut for this rust-lang/rust#58631 to work.
/cc @oli-obk

[spastorino]

@oli-obk this is mergeable now, just force pushed again because there was a conflict.

[mati865]

@spastorino it doesn't build:

error: unexpected end of macro invocation
 --> <::if_chain::__if_chain macros>:6:46
  |
6 | __if_chain ! { @ init ( $ ( $ tt ) * $ head ) $ ( $ tail ) * } } ; (
  |                                              ^ missing tokens in macro arguments
warning: unused imports: `is_copy`, `walk_ptrs_ty_depth`
 --> clippy_lints/src/redundant_clone.rs:2:25
  |
2 |     has_drop, in_macro, is_copy, match_def_path, match_type, paths, snippet_opt, span_lint_node,
  |                         ^^^^^^^
3 |     span_lint_node_and_then, walk_ptrs_ty_depth,
  |                              ^^^^^^^^^^^^^^^^^^
  |
  = note: #[warn(unused_imports)] on by default
warning: unused import: `TerminatorKind`
  --> clippy_lints/src/redundant_clone.rs:13:5
   |
13 |     TerminatorKind,
   |     ^^^^^^^^^^^^^^
error: aborting due to previous error
error: Could not compile `clippy_lints`.

https://travis-ci.com/rust-lang/rust-clippy/jobs/181611795

[mati865]

Suggested change

	if let mir::Operand::Move(mir::Place::Base(PlaceBase(Local(local))) = &args[0];
	if let mir::Operand::Move(mir::Place::Base(PlaceBase(Local(local)))) = &args[0];

It should fix the error.

[spastorino]

@mati865 doh but that's not the issue. PlaceBase::Local is the thing

[mati865]

I admit I haven't checked it deeply, thanks for the explanation.

[flip1995]

@bors r+ p=10

[bors]

📌 Commit 77a67c9 has been approved by flip1995

[bors]

⌛ Testing commit 77a67c9 with merge 8dfabdf...

[bors]

☀️ Test successful - checks-travis, status-appveyor
Approved by: flip1995
Pushing 8dfabdf to master...