resolve/expand: Cache intermediate results of #[derive] expansion

[petrochenkov]

Expansion function for #[derive] (rustc_builtin_macros::derive::Expander::expand) may return an indeterminate result, and therefore can be called multiple times.
Previously we parsed the #[derive(Foo, Bar)]'s input and tried to resolve Foo and Bar on every such call.

Now we maintain a cache Resolver::derive_data and take all the necessary data from it if it was computed previously.
So Foo, Bar is now parsed at most once, and Foo and Bar are successfully resolved at most once.

[rust-highfive]

r? @matthewjasper

(rust-highfive has picked a reviewer for you, use r? to override)

[petrochenkov]

@bors try @rust-timer queue

[rust-timer]

Awaiting bors try build completion.

@rustbot label: +S-waiting-on-perf

[bors]

⌛ Trying commit 45589d2e6e7af5866c804666bf52cbc0e671c2d5 with merge cc63cfbd3047bd17037ca21fa2ac23c7abfe4600...

[bors]

☀️ Try build successful - checks-actions
Build commit: cc63cfbd3047bd17037ca21fa2ac23c7abfe4600 (cc63cfbd3047bd17037ca21fa2ac23c7abfe4600)

[rust-timer]

Queued cc63cfbd3047bd17037ca21fa2ac23c7abfe4600 with parent 1d6b0f6, future comparison URL.

[rust-timer]

Finished benchmarking try commit (cc63cfbd3047bd17037ca21fa2ac23c7abfe4600): comparison url.

Benchmarking this pull request likely means that it is perf-sensitive, so we're automatically marking it as not fit for rolling up. Please note that if the perf results are neutral, you should likely undo the rollup=never given below by specifying rollup- to bors.

Importantly, though, if the results of this run are non-neutral do not roll this PR up -- it will mask other regressions or improvements in the roll up.

@bors rollup=never
@rustbot label: +S-waiting-on-review -S-waiting-on-perf

[petrochenkov]

The results are within the noise, but I'd still like to land this as a refactoring, because it will help with another caching-related FIXME in fn fully_expand_fragment which is more substantial.

[petrochenkov]

It's been almost a month.
r? @Aaron1011

[petrochenkov]

Updated.

[Aaron1011]

@bors r+

[bors]

📌 Commit b965844 has been approved by Aaron1011

[bors]

⌛ Testing commit b965844 with merge 284954f5b9a7702c697585ee899ef91f68ac4dc1...

[rust-log-analyzer]

A job failed! Check out the build log: (web) (plain)

Click to see the possible cause of the failure (guessed by this bot)

[Aaron1011]

Spurious failure
@bors retry

[bors]

💔 Test failed - checks-actions

[bors]

⌛ Testing commit b965844 with merge c755ee4...

[bors]

☀️ Test successful - checks-actions
Approved by: Aaron1011
Pushing c755ee4 to master...