Skip to content
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

Rollup of 7 pull requests #65199

Closed
wants to merge 24 commits into from
Closed

Conversation

Centril
Copy link
Contributor

@Centril Centril commented Oct 8, 2019

Successful merges:

Failed merges:

r? @ghost

Mark-Simulacrum and others added 24 commits October 3, 2019 21:34
When thread sanitizer instrumentation is enabled during compilation of
stack probe function, the function will be miscompiled and trigger
segmentation fault at runtime. Disable stack probes when tsan is
enabled.
`.len()` returns length in bytes so it overestimates the required space
It's now unused, even with -Zquery-dep-graph
Fix the start/end byte positions in the compiler JSON output

Track the changes made during normalization in the `SourceFile` and use this information to correct the `start_byte` and `end_byte` fields in the JSON output.

This should ensure the start/end byte fields can be used to index the original file, even if Rust normalized the source code for parsing purposes. Both CRLF to LF and BOM removal are handled with this one.

The rough plan was discussed with @matklad in rust-lang/rustfix#176 - although I ended up going with `u32` offset tracking so I wouldn't need to deal with `u32 + i32` arithmetics when applying the offset to the span byte positions.

Fixes rust-lang#65029
…chton

Disable stack probe when thread sanitizer is enabled

When thread sanitizer instrumentation is enabled during compilation of
stack probe function, the function will be miscompiled and trigger
segmentation fault at runtime. Disable stack probes when tsan is
enabled.
Correctly estimate the required space for string in `StyledBuffer::prepend`

Fix rust-lang#65119

r? @estebank
…=varkor

When suggesting assoc function with type params, include turbofish

Fix rust-lang#61412, fix rust-lang#61411.
…p, r=michaelwoerister

Remove loaded_from_cache map from DepGraph

It's now unused, even with -Zquery-dep-graph

From https://github.com/rust-lang/rust/pull/63756/files#r316039379 -- it'll simplify that PR to get this landed separately so we can just remove some of the code that it touches.

r? @Zoxc or @michaelwoerister
…aelwoerister

Remove query-related macros

The query system has a few macros that only have one or two call sites, and I find they hurt readability. This PR removes them.

r? @michaelwoerister
@Centril
Copy link
Contributor Author

Centril commented Oct 8, 2019

@bors r+ p=7 rollup=never

@bors
Copy link
Contributor

bors commented Oct 8, 2019

📌 Commit 8c2e726 has been approved by Centril

@bors bors added the S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. label Oct 8, 2019
@bors
Copy link
Contributor

bors commented Oct 8, 2019

⌛ Testing commit 8c2e726 with merge 5fc9c1480e71050291421142b7a899a320713938...

@bors
Copy link
Contributor

bors commented Oct 8, 2019

💔 Test failed - checks-azure

@rust-highfive
Copy link
Collaborator

The job x86_64-gnu-aux of your PR failed (pretty log, raw log). Through arcane magic we have determined that the following fragments from the build log may contain information about the problem.

Click to expand the log.
2019-10-08T13:37:57.1749985Z note: we would appreciate a bug report: https://github.com/rust-lang/rust/blob/master/CONTRIBUTING.md#bug-reports
2019-10-08T13:37:57.1750134Z 
2019-10-08T13:37:57.1750476Z note: rustc 1.40.0-dev running on x86_64-unknown-linux-gnu
2019-10-08T13:37:57.1750592Z 
2019-10-08T13:37:57.1751154Z note: compiler flags: -Z external-macro-backtrace -Z unstable-options -Z binary-dep-depinfo -Z force-unstable-if-unmarked -C opt-level=2 -C debuginfo=0 -C link-args=-Wl,-rpath,$ORIGIN/../lib -C prefer-dynamic -C debug-assertions=y --crate-type lib
2019-10-08T13:37:57.1751417Z note: some of the compiler flags provided by cargo are hidden
2019-10-08T13:37:57.1751495Z 
2019-10-08T13:37:57.2667177Z [RUSTC-TIMING] rustc test:false 0.874
2019-10-08T13:37:57.2667690Z error: could not compile `rustc`.
2019-10-08T13:37:57.2667690Z error: could not compile `rustc`.
2019-10-08T13:37:57.2668064Z warning: build failed, waiting for other jobs to finish...
2019-10-08T13:39:44.6167192Z [RUSTC-TIMING] syntax test:false 169.689
2019-10-08T13:39:44.6277126Z error: build failed
2019-10-08T13:39:44.6300673Z command did not execute successfully: "/checkout/obj/build/x86_64-unknown-linux-gnu/stage0/bin/cargo" "build" "-Zconfig-profile" "--target" "x86_64-unknown-linux-gnu" "-Zbinary-dep-depinfo" "-j" "2" "--release" "--locked" "--color" "always" "--features" "" "--manifest-path" "/checkout/src/rustc/Cargo.toml" "--message-format" "json-render-diagnostics"
2019-10-08T13:39:44.6301020Z expected success, got: exit code: 101
2019-10-08T13:39:44.6320816Z failed to run: /checkout/obj/build/bootstrap/debug/bootstrap test src/test/run-fail/pretty src/test/run-pass-valgrind/pretty src/tools/cargo src/tools/cargotest
2019-10-08T13:39:44.6321030Z Build completed unsuccessfully in 0:45:54
2019-10-08T13:39:44.6377871Z Makefile:50: recipe for target 'check-aux' failed
2019-10-08T13:39:44.6379143Z make: *** [check-aux] Error 1
2019-10-08T13:39:44.6404435Z   local time: Tue Oct  8 13:39:44 UTC 2019
2019-10-08T13:39:44.7954209Z   network time: Tue, 08 Oct 2019 13:39:44 GMT
2019-10-08T13:39:44.7954426Z == end clock drift check ==
2019-10-08T13:39:44.7954426Z == end clock drift check ==
2019-10-08T13:39:46.0090261Z ##[error]Bash exited with code '2'.
2019-10-08T13:39:46.0147691Z ##[section]Starting: Upload CPU usage statistics
2019-10-08T13:39:46.0157924Z ==============================================================================
2019-10-08T13:39:46.0158083Z Task         : Bash
2019-10-08T13:39:46.0158188Z Description  : Run a Bash script on macOS, Linux, or Windows

I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact @TimNN. (Feature Requests)

@bors bors added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. and removed S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. labels Oct 8, 2019
@Centril Centril closed this Oct 8, 2019
@Centril Centril deleted the rollup-tzhip6c branch October 8, 2019 13:44
@Centril Centril added the rollup A PR which is a rollup label Oct 24, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
rollup A PR which is a rollup S-waiting-on-review Status: Awaiting review from the assignee but also interested parties.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants