-
Notifications
You must be signed in to change notification settings - Fork 12.8k
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
Serialize span hygiene data #72121
Merged
Merged
Serialize span hygiene data #72121
Commits on Jul 26, 2020
-
Configuration menu - View commit details
-
Copy full SHA for 0caebfa - Browse repository at this point
Copy the full SHA 0caebfaView commit details -
Configuration menu - View commit details
-
Copy full SHA for 5cd1b5d - Browse repository at this point
Copy the full SHA 5cd1b5dView commit details -
Add test for serializing hygiene *into* a proc-macro crate
This is a very obscure corner case, and should never be hit in practice.
Configuration menu - View commit details
-
Copy full SHA for 979dd77 - Browse repository at this point
Copy the full SHA 979dd77View commit details -
Ignore
orig_id
for stable hashingThis does not affect semantic equality, and was causing an enormous number of Span hash invalidations.
Configuration menu - View commit details
-
Copy full SHA for dc21612 - Browse repository at this point
Copy the full SHA dc21612View commit details -
Configuration menu - View commit details
-
Copy full SHA for 7e0d3fd - Browse repository at this point
Copy the full SHA 7e0d3fdView commit details -
Configuration menu - View commit details
-
Copy full SHA for cf7bef3 - Browse repository at this point
Copy the full SHA cf7bef3View commit details -
Configuration menu - View commit details
-
Copy full SHA for f622f45 - Browse repository at this point
Copy the full SHA f622f45View commit details -
Remove explicit
extern crate
from proc-macro testWe only want to load this auxiliary crate from a proc-macro, so that it only ever needs to get built for the host platform.
Configuration menu - View commit details
-
Copy full SHA for 768803c - Browse repository at this point
Copy the full SHA 768803cView commit details
Commits on Jul 27, 2020
-
Normalize the test output of hygiene-related tests
A raw SyntaxContext id is implicitly dependent on the target platform, since libstd and libcore have platform-dependent #[cfg]s which affect which macros are invoked. As a result, we must strip out any SyntaxContext ids from test output to ensure that the captured stdout is not platform-dependent.
Configuration menu - View commit details
-
Copy full SHA for f7235a8 - Browse repository at this point
Copy the full SHA f7235a8View commit details
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.