rustc_metadata: don't break the version check when CrateRoot changes.#37931
Merged
bors merged 1 commit intorust-lang:masterfrom Nov 24, 2016
Merged
rustc_metadata: don't break the version check when CrateRoot changes.#37931bors merged 1 commit intorust-lang:masterfrom
bors merged 1 commit intorust-lang:masterfrom
Conversation
Contributor
|
r? @arielb1 (rust_highfive has picked a reviewer for you, use r? to override) |
Member
Author
|
r? @jseyfried |
Collaborator
|
📌 Commit f4c68d2 has been approved by |
Collaborator
|
⌛ Testing commit f4c68d2 with merge 1e45b63... |
bors
added a commit
that referenced
this pull request
Nov 23, 2016
rustc_metadata: don't break the version check when CrateRoot changes. In #36551 I made `rustc_version` a field of `CrateRoot`, but despite it being the first field, one could still break the version check by changing `CrateRoot` so older compilers couldn't fully decode it (e.g. #37463). This PR fixes #37803 by moving the version string back at the beginning of metadata, right after the 32-bit big-endian absolute position of `CrateRoot`, and by incrementing `METADATA_VERSION`.
Collaborator
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
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.
In #36551 I made
rustc_versiona field ofCrateRoot, but despite it being the first field, one could still break the version check by changingCrateRootso older compilers couldn't fully decode it (e.g. #37463).This PR fixes #37803 by moving the version string back at the beginning of metadata, right after the 32-bit big-endian absolute position of
CrateRoot, and by incrementingMETADATA_VERSION.