Skip to content

Split up infer.rs further#2004

Merged
bors[bot] merged 1 commit intorust-lang:masterfrom
flodiebold:infer-split
Oct 12, 2019
Merged

Split up infer.rs further#2004
bors[bot] merged 1 commit intorust-lang:masterfrom
flodiebold:infer-split

Conversation

@flodiebold
Copy link
Copy Markdown
Member

  • coercion logic goes to coerce.rs
  • expression inference goes to expr.rs
  • pattern inference goes to pat.rs

Copy link
Copy Markdown
Contributor

@matklad matklad left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGMT!

I wonder if we should try to split the tests as well... Not sure what's the best module structure around it though

Comment thread crates/ra_hir/src/ty/infer/coerce.rs Outdated
Comment thread crates/ra_hir/src/ty/infer/coerce.rs
Comment thread crates/ra_hir/src/ty/infer/pat.rs
 - coercion logic goes to `coerce.rs`
 - expression inference goes to `expr.rs`
 - pattern inference goes to `pat.rs`
@flodiebold
Copy link
Copy Markdown
Member Author

bors r+

bors Bot added a commit that referenced this pull request Oct 12, 2019
2004: Split up infer.rs further r=flodiebold a=flodiebold

 - coercion logic goes to `coerce.rs`
 - expression inference goes to `expr.rs`
 - pattern inference goes to `pat.rs`

Co-authored-by: Florian Diebold <flodiebold@gmail.com>
@bors
Copy link
Copy Markdown
Contributor

bors Bot commented Oct 12, 2019

Build succeeded

@bors bors Bot merged commit 2955fbc into rust-lang:master Oct 12, 2019
@flodiebold flodiebold deleted the infer-split branch October 12, 2019 22:55
github-actions Bot pushed a commit that referenced this pull request Feb 16, 2026
aarch64: use `read_unaligned` for `vld1_*`
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants