Diagnostic information does not include item attributes

[jrobsonchase]

Original rustfix issue: rust-lang/rustfix#147

When rustc outputs warnings like "you don't need this extern crate line anymore, delete it," it doesn't include accompanying attributes. This leads rustfix to apply its suggestion to delete the extern crate line, but leave the attribute, which then gets applied to the next item.

For example:

#[cfg(unix)]
extern crate nix;

mod foo;

becomes

#[cfg(unix)]

mod foo;

which applies the #[cfg(unix)] attribute to mod foo; which is definitely not what was desired.

[zackmdavis]

Here's where we issue the diagnostic:

rust/src/librustc_typeck/check_unused.rs

Lines 137 to 152 in 2fa1390

	if extern_crate.warn_if_unused {
	if let Some(&span) = unused_extern_crates.get(&extern_crate.def_id) {
	assert_eq!(extern_crate.def_id.krate, LOCAL_CRATE);
	let hir_id = tcx.hir.definitions().def_index_to_hir_id(extern_crate.def_id.index);
	let id = tcx.hir.hir_to_node_id(hir_id);
	let msg = "unused extern crate";
	tcx.struct_span_lint_node(lint, id, span, msg)
	.span_suggestion_short_with_applicability(
	span,
	"remove it",
	String::new(),
	Applicability::MachineApplicable)
	.emit();
	continue;
	}
	}

Assuming the cfg attribute is actually preserved by earlier passes, then correcting for this case should be easy (look for it in tcx.get_attrs and extend the span correspondingly), but I don't remember offhand whether that's true and don't have time to investigate right now. If it isn't, then this is similar-in-spirit to #45216 in that we "just" want the span of an attribute, but would need to do some nontrivial rearchitecting work to make it available.