Skip to content

Remove special case for ExprKind::Paren in MutVisitor - #87284

Merged
bors merged 1 commit into
rust-lang:masterfrom
Aaron1011:remove-paren-special
Jul 20, 2021
Merged

Remove special case for ExprKind::Paren in MutVisitor#87284
bors merged 1 commit into
rust-lang:masterfrom
Aaron1011:remove-paren-special

Conversation

@Aaron1011

@Aaron1011Aaron1011 commented Jul 19, 2021

Copy link
Copy Markdown
Contributor

The special case breaks several useful invariants (ExpnIds are
globally unique, and never change). This special case
was added back in 2016 in #34355

r? @petrochenkov

@rust-highfiverust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jul 19, 2021
@Aaron1011

Aaron1011 commented Jul 19, 2021

Copy link
Copy Markdown
ContributorAuthor

Fixes #87274

@Aaron1011

Aaron1011 commented Jul 19, 2021

Copy link
Copy Markdown
ContributorAuthor

@bors try @rust-timer queue

@rust-timer

rust-timer commented Jul 19, 2021

Copy link
Copy Markdown
Collaborator

Awaiting bors try build completion.

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

@rustbotrustbot added the S-waiting-on-perf Status: Waiting on a perf run to be completed. label Jul 19, 2021
@bors

bors commented Jul 19, 2021

Copy link
Copy Markdown
Collaborator

⌛ Trying commit 208bc2d83fbcd478aacbd62b4f6f9f18f3d4379e with merge 2f692747360443b2e47774330701e5705186fcec...

@bors

bors commented Jul 19, 2021

Copy link
Copy Markdown
Collaborator

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

@rust-timer

rust-timer commented Jul 19, 2021

Copy link
Copy Markdown
Collaborator

Queued 2f692747360443b2e47774330701e5705186fcec with parent 8df945c, future comparison URL.

@petrochenkovpetrochenkov removed the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jul 19, 2021
@rust-timer

rust-timer commented Jul 19, 2021

Copy link
Copy Markdown
Collaborator

Finished benchmarking try commit (2f692747360443b2e47774330701e5705186fcec): comparison url.

Summary: This benchmark run did not return any significant changes.

If you disagree with this performance assessment, please file an issue in rust-lang/rustc-perf.

Benchmarking this pull request likely means that it is perf-sensitive, so we're automatically marking it as not fit for rolling up.

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

@rustbotrustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. and removed S-waiting-on-perf Status: Waiting on a perf run to be completed. labels Jul 19, 2021
@petrochenkov

petrochenkov commented Jul 19, 2021

Copy link
Copy Markdown
Contributor

@bors r+ rollup=maybe

@bors

bors commented Jul 19, 2021

Copy link
Copy Markdown
Collaborator

📌 Commit 208bc2d83fbcd478aacbd62b4f6f9f18f3d4379e has been approved by petrochenkov

@borsbors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Jul 19, 2021
@petrochenkov

petrochenkov commented Jul 19, 2021

Copy link
Copy Markdown
Contributor

Oh, wait, could you fix the typo ExpnKind -> ExprKind in the commit and PR titles?
r=me after that.
@bors r-

@borsbors added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. and removed S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. labels Jul 19, 2021
@Aaron1011
Remove special case for ExprKind::Paren in MutVisitor
f9f238e
The special case breaks several useful invariants (`ExpnId`s are
globally unique, and never change). This special case
was added back in 2016 in rust-lang#34355
@Aaron1011
Aaron1011force-pushed the remove-paren-special branch from 208bc2d to f9f238eCompareJuly 19, 2021 22:23
@Aaron1011Aaron1011 changed the title Remove special case for ExpnKind::Paren in MutVisitorRemove special case for ExprKind::Paren in MutVisitorJul 19, 2021
@Aaron1011

Aaron1011 commented Jul 19, 2021

Copy link
Copy Markdown
ContributorAuthor

@bors r=petrochenkov p=1

This unbreaks a perf benchmark

@bors

bors commented Jul 19, 2021

Copy link
Copy Markdown
Collaborator

📌 Commit f9f238e has been approved by petrochenkov

@borsbors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. labels Jul 19, 2021
@bors

bors commented Jul 19, 2021

Copy link
Copy Markdown
Collaborator

⌛ Testing commit f9f238e with merge 6535449...

@bors

bors commented Jul 20, 2021

Copy link
Copy Markdown
Collaborator

☀️ Test successful - checks-actions
Approved by: petrochenkov
Pushing 6535449 to master...

@borsbors added the merged-by-bors This PR was explicitly merged by bors. label Jul 20, 2021
@bors
bors merged commit 6535449 into rust-lang:masterJul 20, 2021
@rustbotrustbot added this to the 1.55.0 milestone Jul 20, 2021
This was referenced Jul 20, 2021
Closed
Closed
@mockersfmockersf mentioned this pull request Jul 20, 2021
Closed
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@petrochenkovpetrochenkov

Labels

merged-by-borsThis PR was explicitly merged by bors.S-waiting-on-borsStatus: Waiting on bors to run and complete tests. Bors will change the label on completion.

Projects

None yet

Milestone

1.55.0

Development

Successfully merging this pull request may close these issues.

6 participants

@Aaron1011@rust-timer@bors@petrochenkov@rust-highfive@rustbot