Rollup of 5 pull requests #64583
Merged
Rollup of 5 pull requests #64583
Conversation
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
Use the new dataflow cursor.
Make rustc_mir::dataflow module pub (for clippy) I'm working on fixing false-positives of a MIR-based clippy lint (rust-lang/rust-clippy#4509), and in need of the dataflow infrastructure.
…p_hint, r=alexcrichton PR: documentation spin loop hint The documentation for 'spin loop hint' explains that yield is better if the lock holder is running on the same CPU. I suggest that 'CPU or core' would be clearer.
… r=tmandry Replace `state_for_location` with `DataflowResultsCursor` These are two different ways of getting the same data from the result of a dataflow analysis. However, `state_for_location` goes quadratic if you try to call it for every statement in the body.
Fix issue22656 with LLDB 8 Closes rust-lang#64074
Update books ## book 24 commits in 7ddc46460f09a5cd9bd2a620565bdc20b3315ea9..871416b85c1a73717d65d6f4a9ea29e5aef3db0e 2019-06-27 09:50:36 -0400 to 2019-09-16 09:46:20 -0400 - Ch16-2 add missing Ferris (rust-lang/book#2033) - Update version mentioned on the front page - Update error messages (rust-lang/book#1737) - Update version of Rust used to 1.37 - Replace Cargo docs link with a more specific link (rust-lang/book#2066) - Added missing await reserved keyword (rust-lang/book#2064) - add does_not_compile for a snippet (rust-lang/book#2056) - Added second missing dyn (rust-lang/book#2046) - Removed unnecessary & in function call (rust-lang/book#2038) - Printing non-Display structs is a *compile* error (rust-lang/book#2031) - Update Readme mdBook version to match linked file (rust-lang/book#2012) - Update loose mdbook version reference (rust-lang/book#2003) - Added a bullet point to have list of things unsafe allows for match u… (rust-lang/book#1993) - Rewrote a confusing sentence (rust-lang/book#1986) - Replace deprecated `...` range syntax with `..=` (rust-lang/book#1977) - correct wording for integration test doc (rust-lang/book#1971) - Mark the dangle function as does_not_compile (rust-lang/book#1965) - Add more words to the quote from the actual Go documentation (rust-lang/book#1960) - Remove unused import in lfp (rust-lang/book#1944) - A small typo? (rust-lang/book#1931) - Make the code not compile to match the text (rust-lang/book#1926) - Ferris does-not-compile added (ch9.2) (rust-lang/book#1925) - ch07 - remove note regarding use and relative path (rust-lang/book#1820) - tweak opening paragraph of deref coercions (rust-lang/book#1749) ## rust-by-example 9 commits in e76be6b2dc84c6a992e186157efe29d625e29b94..67cfbf31df880728dcf7cb35b15b028ec92caf31 2019-09-03 07:42:26 -0300 to 2019-09-18 09:36:40 -0300 - Fix rust-lang/rust-by-example#90: Add supertraits and Fully Qualified syntax (rust-lang/rust-by-example#1259) - Fix some broken links. (rust-lang/rust-by-example#1258) - Fix rust-lang/rust-by-example#1253: Document enum type aliases (rust-lang/rust-by-example#1255) - Inline code in some new/changed chapters (rust-lang/rust-by-example#1254) - fix rust-lang/rust-by-example#1067: explain that unit tests can return Result<()> (rust-lang/rust-by-example#1252) - Fix rust-lang/rust-by-example#1060: add page on Impl Trait (rust-lang/rust-by-example#1251) - Fix rust-lang/rust-by-example#1053: Added a page about the dyn keyword (rust-lang/rust-by-example#1249) - Fix rust-lang/rust-by-example#1110: add examples of ? and Option (rust-lang/rust-by-example#1250) - fix 1037: add the TryFrom chapter back in (rust-lang/rust-by-example#1247)
|
@bors r+ p=5 |
|
📌 Commit eeda313 has been approved by |
bors
added
the
S-waiting-on-bors
bors
added a commit
that referenced
this pull request
Sep 18, 2019
Rollup of 5 pull requests Successful merges: - #64207 (Make rustc_mir::dataflow module pub (for clippy)) - #64348 (PR: documentation spin loop hint) - #64532 (Replace `state_for_location` with `DataflowResultsCursor`) - #64578 (Fix issue22656 with LLDB 8) - #64580 (Update books) Failed merges: r? @ghost
|
☀️ Test successful - checks-azure |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Successful merges:
state_for_locationwithDataflowResultsCursor#64532 (Replacestate_for_locationwithDataflowResultsCursor)Failed merges:
r? @ghost