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

This commit moves RISC-V from the experimental LLVM targets to the
regular LLVM targets. RISC-V was made non-experimental in
https://reviews.llvm.org/rL366399

I have also sorted the list of LLVM targets, and changed the code
around setting llvm_exp_targets (and its default) to match the code
setting llvm_targets (and its default), ensuring future changes to
the defaults, as LLVM targets become stable, affect as few places as
possible.

…nsupported cases

…ive paths

Fix typo in Delimited::open_tt

…chenkov

Cleanup syntax::attr

Mostly removing needless arguments to constructors

r? @petrochenkov

…excrichton

rustbuild: RISC-V is no longer an experimental LLVM target

This moves RISC-V from the experimental LLVM targets to the
regular LLVM targets. RISC-V was made non-experimental in
https://reviews.llvm.org/rL366399

I have also sorted the list of LLVM targets, and changed the code
around setting llvm_exp_targets (and its default) to match the code
setting llvm_targets (and its default), ensuring future changes to
the defaults, as LLVM targets become stable, affect as few places as
possible.

Given WebAssembly is in `LLVM_ALL_TARGETS` and is therefore built by default (and has been since October 2018), I'm not sure why rust still has it in `experimental-targets`. I'm happy to update this PR to move it into the main list of LLVM targets.

r? @alexcrichton

dead_code: Properly inspect fields in struct patterns with type relative paths

Closes rust-lang#63151.

A bit of Miri error cleanup

Some cleanup after rust-lang#62969.

r? @oli-obk
Cc @saleemjaffer