Move some queries from rustc::ty to librustc_ty. #67780

cjgillot · 2020-01-01T19:05:47Z

rust-highfive · 2020-01-01T19:05:51Z

(rust_highfive has picked a reviewer for you, use r? to override)

Zoxc · 2020-01-01T20:30:19Z

I think it would make sense to move this to a new rustc_ty crate. cc @rust-lang/compiler

bors · 2020-01-02T07:06:36Z

☔ The latest upstream changes (presumably #67700) made this pull request unmergeable. Please resolve the merge conflicts.

rust-highfive · 2020-01-05T16:24:16Z

The job x86_64-gnu-llvm-7 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.

2020-01-05T15:57:40.8847512Z ##[command]git remote add origin https://github.com/rust-lang/rust
2020-01-05T15:57:40.9074951Z ##[command]git config gc.auto 0
2020-01-05T15:57:40.9140058Z ##[command]git config --get-all http.https://github.com/rust-lang/rust.extraheader
2020-01-05T15:57:40.9203369Z ##[command]git config --get-all http.proxy
2020-01-05T15:57:40.9351315Z ##[command]git -c http.extraheader="AUTHORIZATION: basic ***" fetch --force --tags --prune --progress --no-recurse-submodules --depth=2 origin +refs/heads/*:refs/remotes/origin/* +refs/pull/67780/merge:refs/remotes/pull/67780/merge
---
2020-01-05T16:24:01.2867426Z Assembling stage1 compiler (x86_64-unknown-linux-gnu)
2020-01-05T16:24:01.2886861Z Building stage1 std artifacts (x86_64-unknown-linux-gnu -> x86_64-unknown-linux-gnu)
2020-01-05T16:24:01.6879880Z    Compiling cc v1.0.47
2020-01-05T16:24:01.6880214Z    Compiling core v0.0.0 (/checkout/src/libcore)
2020-01-05T16:24:06.4571088Z error: internal compiler error: src/librustc/ty/query/mod.rs:96: `tcx.associated_item_def_ids(DefId(0:5925 ~ core[a1e0]::fmt[0]::Debug[0]))` unsupported by its crate
2020-01-05T16:24:06.4572342Z thread 'rustc' panicked at 'Box<Any>', src/librustc_errors/lib.rs:892:9
2020-01-05T16:24:06.4572439Z note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace.
2020-01-05T16:24:06.4572626Z 
2020-01-05T16:24:06.4577822Z note: the compiler unexpectedly panicked. this is a bug.
2020-01-05T16:24:06.4577822Z note: the compiler unexpectedly panicked. this is a bug.
2020-01-05T16:24:06.4577956Z 
2020-01-05T16:24:06.4579060Z note: we would appreciate a bug report: ***/blob/master/CONTRIBUTING.md#bug-reports
2020-01-05T16:24:06.4579329Z 
2020-01-05T16:24:06.4580848Z note: rustc 1.42.0-nightly (179694ba8 2020-01-05) running on x86_64-unknown-linux-gnu
2020-01-05T16:24:06.4580895Z 
2020-01-05T16:24:06.4581420Z note: compiler flags: -Z external-macro-backtrace -Z binary-dep-depinfo -Z force-unstable-if-unmarked -C opt-level=2 -C codegen-units=1 -C debuginfo=0 -C link-args=-Wl,-rpath,$ORIGIN/../lib -C prefer-dynamic -C llvm-args=-import-instr-limit=10 -C debug-assertions=n --crate-type lib
2020-01-05T16:24:06.4581549Z note: some of the compiler flags provided by cargo are hidden
2020-01-05T16:24:06.4581581Z 
2020-01-05T16:24:06.4851204Z error: aborting due to previous error
2020-01-05T16:24:06.4851436Z 
---
2020-01-05T16:24:07.7672224Z   local time: Sun Jan  5 16:24:07 UTC 2020
2020-01-05T16:24:08.1820397Z   network time: Sun, 05 Jan 2020 16:24:08 GMT
2020-01-05T16:24:08.1831333Z == end clock drift check ==
2020-01-05T16:24:10.4063622Z 
2020-01-05T16:24:10.4161739Z ##[error]Bash exited with code '1'.
2020-01-05T16:24:10.4203578Z ##[section]Starting: Checkout
2020-01-05T16:24:10.4205759Z ==============================================================================
2020-01-05T16:24:10.4205820Z Task         : Get sources
2020-01-05T16:24:10.4205868Z Description  : Get sources from a repository. Supports Git, TfsVC, and SVN repositories.

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)

estebank · 2020-01-05T21:43:54Z

src/librustc_passes/ty.rs

+//pub fn is_impl_trait_defn(tcx: TyCtxt<'_>, def_id: DefId) -> Option<DefId> {
+//    if let Some(hir_id) = tcx.hir().as_local_hir_id(def_id) {
+//        if let Node::Item(item) = tcx.hir().get(hir_id) {
+//            if let hir::ItemKind::OpaqueTy(ref opaque_ty) = item.kind {
+//                return opaque_ty.impl_trait_fn;
+//            }
+//        }
+//    }
+//    None
+//}


bors · 2020-01-06T16:09:36Z

☔ The latest upstream changes (presumably #67886) made this pull request unmergeable. Please resolve the merge conflicts.

matthewjasper · 2020-01-07T12:51:53Z

Nominating for discussion of rustc_ty

bors · 2020-01-08T22:58:07Z

☔ The latest upstream changes (presumably #68011) made this pull request unmergeable. Please resolve the merge conflicts.

Zoxc · 2020-01-13T13:01:33Z

@bors r+

bors · 2020-01-13T13:01:35Z

📌 Commit 9908a87 has been approved by Zoxc

rust-highfive · 2020-01-13T13:09:09Z

The job x86_64-gnu-llvm-7 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.

2020-01-13T12:42:20.1002348Z ##[command]git remote add origin https://github.com/rust-lang/rust
2020-01-13T12:42:20.1108938Z ##[command]git config gc.auto 0
2020-01-13T12:42:20.1164327Z ##[command]git config --get-all http.https://github.com/rust-lang/rust.extraheader
2020-01-13T12:42:20.1236961Z ##[command]git config --get-all http.proxy
2020-01-13T12:42:20.1401045Z ##[command]git -c http.extraheader="AUTHORIZATION: basic ***" fetch --force --tags --prune --progress --no-recurse-submodules --depth=2 origin +refs/heads/*:refs/remotes/origin/* +refs/pull/67780/merge:refs/remotes/pull/67780/merge
---
2020-01-13T13:08:55.3577288Z Assembling stage1 compiler (x86_64-unknown-linux-gnu)
2020-01-13T13:08:55.3612431Z Building stage1 std artifacts (x86_64-unknown-linux-gnu -> x86_64-unknown-linux-gnu)
2020-01-13T13:08:55.6482075Z    Compiling cc v1.0.49
2020-01-13T13:08:55.6521563Z    Compiling core v0.0.0 (/checkout/src/libcore)
2020-01-13T13:09:00.3138950Z error: internal compiler error: src/librustc/ty/query/mod.rs:99: `tcx.associated_item_def_ids(DefId(0:5917 ~ core[6099]::fmt[0]::Debug[0]))` unsupported by its crate
2020-01-13T13:09:00.3140422Z thread 'rustc' panicked at 'Box<Any>', src/librustc_errors/lib.rs:872:9
2020-01-13T13:09:00.3140854Z note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace.
2020-01-13T13:09:00.3141088Z 
2020-01-13T13:09:00.3144603Z note: the compiler unexpectedly panicked. this is a bug.
2020-01-13T13:09:00.3144603Z note: the compiler unexpectedly panicked. this is a bug.
2020-01-13T13:09:00.3146771Z 
2020-01-13T13:09:00.3147956Z note: we would appreciate a bug report: ***/blob/master/CONTRIBUTING.md#bug-reports
2020-01-13T13:09:00.3148232Z 
2020-01-13T13:09:00.3148892Z note: rustc 1.42.0-nightly (d4d5b3ad7 2020-01-13) running on x86_64-unknown-linux-gnu
2020-01-13T13:09:00.3149092Z 
2020-01-13T13:09:00.3149750Z note: compiler flags: -Z external-macro-backtrace -Z binary-dep-depinfo -Z force-unstable-if-unmarked -C opt-level=2 -C codegen-units=1 -C debuginfo=0 -C link-args=-Wl,-rpath,$ORIGIN/../lib -C prefer-dynamic -C llvm-args=-import-instr-limit=10 -C debug-assertions=n --crate-type lib
2020-01-13T13:09:00.3153404Z note: some of the compiler flags provided by cargo are hidden
2020-01-13T13:09:00.3153732Z 
2020-01-13T13:09:00.3403081Z error: aborting due to previous error
2020-01-13T13:09:00.3403761Z 
---
2020-01-13T13:09:01.8252248Z   local time: Mon Jan 13 13:09:01 UTC 2020
2020-01-13T13:09:02.1205336Z   network time: Mon, 13 Jan 2020 13:09:02 GMT
2020-01-13T13:09:02.1205453Z == end clock drift check ==
2020-01-13T13:09:03.6562252Z 
2020-01-13T13:09:03.6670261Z ##[error]Bash exited with code '1'.
2020-01-13T13:09:03.6925005Z ##[section]Starting: Checkout
2020-01-13T13:09:03.6927267Z ==============================================================================
2020-01-13T13:09:03.6927331Z Task         : Get sources
2020-01-13T13:09:03.6927384Z Description  : Get sources from a repository. Supports Git, TfsVC, and SVN repositories.

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)

Zoxc · 2020-01-13T13:16:14Z

@bors r-

Zoxc · 2020-01-13T13:17:57Z

src/librustc_ty/ty.rs

+    fn_like.asyncness()
+}
+
+pub fn provide(providers: &mut ty::query::Providers<'_>) {


I guess you need to call this.

Oops, sorry. Added a commit for that.

Zoxc · 2020-01-13T23:57:16Z

@bors r+

bors · 2020-01-13T23:57:18Z

📌 Commit 894dc2d has been approved by Zoxc

Move some queries from rustc::ty to librustc_ty. cc rust-lang#65031

@ghost

Rollup of 5 pull requests Successful merges: - #67780 (Move some queries from rustc::ty to librustc_ty.) - #68096 (Clean up some diagnostics by making them more consistent) - #68223 (Use 3.6 instead of 3.5 in float fract() documentation) - #68265 (Fix some issue numbers of unstable features) - #68266 (Changed docs for f32 and f64.) Failed merges: - #68204 (Use named fields for `{ast,hir}::ItemKind::Impl`) r? @ghost

rust-highfive assigned varkor Jan 1, 2020

rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jan 1, 2020

Zoxc self-assigned this Jan 2, 2020

cjgillot force-pushed the passes-ty branch from 9552051 to 2d9baa1 Compare January 5, 2020 15:56

estebank reviewed Jan 5, 2020

View reviewed changes

cjgillot force-pushed the passes-ty branch 2 times, most recently from f86b7d8 to 1be4d25 Compare January 6, 2020 08:15

cjgillot mentioned this pull request Jan 7, 2020

Split librustc::{traits,infer} to a separate crate rustc_infer #67953

Merged

matthewjasper added the I-nominated label Jan 7, 2020

cjgillot force-pushed the passes-ty branch from 1be4d25 to c08dbb0 Compare January 7, 2020 21:27

Move some queries from rustc::ty to librustc_passes.

6d5170c

cjgillot force-pushed the passes-ty branch from c08dbb0 to 6d5170c Compare January 11, 2020 18:01

Move to new crate rustc_ty.

9908a87

bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Jan 13, 2020

bors added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. and removed S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. labels Jan 13, 2020

Zoxc reviewed Jan 13, 2020

View reviewed changes

Do not forget to provide queries.

894dc2d

cjgillot changed the title ~~Move some queries from rustc::ty to librustc_passes.~~ Move some queries from rustc::ty to librustc_ty. Jan 13, 2020

bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. labels Jan 13, 2020

Dylan-DPC-zz pushed a commit to Dylan-DPC-zz/rust that referenced this pull request Jan 16, 2020

Rollup merge of rust-lang#67780 - cjgillot:passes-ty, r=Zoxc

6b069d6

Move some queries from rustc::ty to librustc_ty. cc rust-lang#65031

Dylan-DPC-zz mentioned this pull request Jan 16, 2020

Rollup of 6 pull requests #68271

Closed

Dylan-DPC-zz pushed a commit to Dylan-DPC-zz/rust that referenced this pull request Jan 16, 2020

Rollup merge of rust-lang#67780 - cjgillot:passes-ty, r=Zoxc

6b83862

Move some queries from rustc::ty to librustc_ty. cc rust-lang#65031

Dylan-DPC-zz mentioned this pull request Jan 16, 2020

Rollup of 5 pull requests #68272

Merged

bors merged commit 894dc2d into rust-lang:master Jan 16, 2020

cjgillot deleted the passes-ty branch January 16, 2020 19:43

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Move some queries from rustc::ty to librustc_ty. #67780

Move some queries from rustc::ty to librustc_ty. #67780

cjgillot commented Jan 1, 2020

rust-highfive commented Jan 1, 2020

Zoxc commented Jan 1, 2020

bors commented Jan 2, 2020

rust-highfive commented Jan 5, 2020

estebank Jan 5, 2020

bors commented Jan 6, 2020

matthewjasper commented Jan 7, 2020

bors commented Jan 8, 2020

Zoxc commented Jan 13, 2020

bors commented Jan 13, 2020

rust-highfive commented Jan 13, 2020

Zoxc commented Jan 13, 2020

Zoxc Jan 13, 2020

cjgillot Jan 13, 2020

Zoxc commented Jan 13, 2020

bors commented Jan 13, 2020

Move some queries from rustc::ty to librustc_ty. #67780

Move some queries from rustc::ty to librustc_ty. #67780

Conversation

cjgillot commented Jan 1, 2020

rust-highfive commented Jan 1, 2020

Zoxc commented Jan 1, 2020

bors commented Jan 2, 2020

rust-highfive commented Jan 5, 2020

estebank Jan 5, 2020

Choose a reason for hiding this comment

bors commented Jan 6, 2020

matthewjasper commented Jan 7, 2020

bors commented Jan 8, 2020

Zoxc commented Jan 13, 2020

bors commented Jan 13, 2020

rust-highfive commented Jan 13, 2020

Zoxc commented Jan 13, 2020

Zoxc Jan 13, 2020

Choose a reason for hiding this comment

cjgillot Jan 13, 2020

Choose a reason for hiding this comment

Zoxc commented Jan 13, 2020

bors commented Jan 13, 2020