-
Notifications
You must be signed in to change notification settings - Fork 12.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
rustc: remove dead code #13244
Merged
Merged
rustc: remove dead code #13244
Changes from all commits
Commits
Show all changes
45 commits
Select commit
Hold shift + click to select a range
3b4a517
rustc: make comments on CrateContext doc comments
emberian 5a4c8a9
back: abi: remove dead code
emberian 74ddaa2
back: archive: remove dead code
emberian 59645c1
back: link: remove dead code
emberian aae9594
lib: llvm: remove dead code
emberian 86e6a80
metadata: common: remove dead code
emberian 6ce37e8
metadata: csearch: remove dead code
emberian 3f8d158
metadata: cstore: remove dead code
emberian 0a0ab40
metadata: decoder: remove dead code
emberian 194242f
metadata: filesearch: remove dead code
emberian 0ce17d9
middle: astencode: remove dead code
emberian 77d1978
middle: borrowck: remove dead code
emberian 2013488
middle: cfg: allow dead code
emberian d03e647
middle: const_eval: remove dead code
emberian 4e5409b
middle: dataflow: remove dead code
emberian 6ed7871
middle: freevars: remove dead code
emberian fb19d68
middle: graph: ignore dead code
emberian 35c98a8
middle: kind: remove dead code
emberian 393bc08
middle: lang_items: allow dead code
emberian dc59df7
middle: mem_categorization: remove dead code
emberian 39fce48
middle: pat_util: remove dead code
emberian ee82233
middle: region: remove dead code
emberian a9bf099
middle: trans: adt: remove dead code
emberian 49dc019
middle: trans: base: remove dead code
emberian 8de5eec
middle: trans: build: remove dead code
emberian 5e2b522
middle: trans: builder: ignore dead code
emberian a2290cc
middle: trans: cabi: remove dead code
emberian 8f385fc
middle: trans: callee: remove dead code
emberian e53f48b
middle: trans: common: remove dead code
emberian 9dd185c
middle: trans: context: remove dead code
emberian f997310
middle: trans: datum: remove dead code
emberian 32753ed
middle: trans: meth: remove dead code
emberian 4bc2c3b
middle: trans: reflect: remove dead code
emberian 53b70a8
middle: trans: tvec: remove dead code
emberian b465394
middle: trans: type_: remove dead code
emberian 39f839d
middle: ty: remove dead code
emberian dcb01ff
middle: typeck: check: remove dead code
emberian 1a98ec8
middle: typeck: infer: combine: remove dead code
emberian ce47c91
middle: typeck: infer: lub: remove dead code
emberian 167c29f
middle: typeck: infer: remove dead code
emberian 009389f
middle: typeck: infer: resolve: remove dead code
emberian 3bd0a37
middle: typeck: remove dead code
emberian fdcb104
middle: typeck: rscope: remove dead code
emberian 841eb1c
util: common: remove dead code
emberian 46790a7
util: ppaux: remove dead code
emberian File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should this be made
2u
? (same question for other cases)There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this is used for the offset of the field in the struct. this is hard-coded, and doesn't change with order. if it works today, it will continue to work.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, I know it'll continue to work. It was more a "Do we want this to be sequential?"
Either way works for me, TBH.