Skip to content

Take the def_map argument to TyCtxt::create_and_enter out of its RefCell. - #33977

Merged
bors merged 1 commit into
rust-lang:masterfrom
Ms2ger:create-and-enter
Jun 1, 2016
Merged

Take the def_map argument to TyCtxt::create_and_enter out of its RefCell.#33977
bors merged 1 commit into
rust-lang:masterfrom
Ms2ger:create-and-enter

Conversation

@Ms2ger

@Ms2gerMs2ger commented May 31, 2016

Copy link
Copy Markdown
Contributor

No description provided.

@rust-highfive

rust-highfive commented May 31, 2016

Copy link
Copy Markdown
Contributor

r? @nrc

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

@nrc

nrc commented May 31, 2016

Copy link
Copy Markdown
Member

@bors: r+ rollup

@bors

bors commented May 31, 2016

Copy link
Copy Markdown
Collaborator

📌 Commit 7463c95 has been approved by nrc

@bors

bors commented May 31, 2016

Copy link
Copy Markdown
Collaborator

⌛ Testing commit 7463c95 with merge 49da43e...

bors added a commit that referenced this pull request May 31, 2016
@bors
Auto merge of#33977- Ms2ger:create-and-enter, r=nrc
49da43e
Take the def_map argument to TyCtxt::create_and_enter out of its RefCell.
@bors

bors commented May 31, 2016

Copy link
Copy Markdown
Collaborator

💔 Test failed - auto-win-gnu-32-opt-rustbuild

@alexcrichton

alexcrichton commented May 31, 2016

Copy link
Copy Markdown
Member

@bors: retry

On Tue, May 31, 2016 at 4:18 AM, bors notifications@github.com wrote:

💔 Test failed - auto-win-gnu-32-opt-rustbuild
http://buildbot.rust-lang.org/builders/auto-win-gnu-32-opt-rustbuild/builds/1303

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
#33977 (comment), or mute
the thread
https://github.com/notifications/unsubscribe/AAD95DfVKcZRtRScFtuGcBqVl-iAWB2Jks5qHBj1gaJpZM4IqPqp
.

@bors

bors commented May 31, 2016

Copy link
Copy Markdown
Collaborator

⚡ Previous build results for auto-linux-64-cargotest, auto-linux-cross-opt, auto-win-msvc-64-cargotest are reusable. Rebuilding only auto-linux-32-nopt-t, auto-linux-32-opt, auto-linux-32cross-opt, auto-linux-64-cross-armhf, auto-linux-64-cross-armsf, auto-linux-64-cross-freebsd, auto-linux-64-cross-netbsd, auto-linux-64-debug-opt, auto-linux-64-nopt-t, auto-linux-64-opt, auto-linux-64-opt-mir, auto-linux-64-opt-rustbuild, auto-linux-64-x-android-t, auto-linux-musl-64-opt, auto-mac-32-opt, auto-mac-64-nopt-t, auto-mac-64-opt, auto-mac-64-opt-rustbuild, auto-mac-cross-ios-opt, auto-win-gnu-32-nopt-t, auto-win-gnu-32-opt, auto-win-gnu-32-opt-rustbuild, auto-win-gnu-64-nopt-t, auto-win-gnu-64-opt, auto-win-msvc-32-cross-opt, auto-win-msvc-32-opt, auto-win-msvc-64-opt, auto-win-msvc-64-opt-mir, auto-win-msvc-64-opt-rustbuild...

@bors

bors commented May 31, 2016

Copy link
Copy Markdown
Collaborator

💔 Test failed - auto-linux-64-opt-mir

@alexcrichton

alexcrichton commented May 31, 2016

Copy link
Copy Markdown
Member

@bors: retry

On Tue, May 31, 2016 at 4:02 PM, bors notifications@github.com wrote:

💔 Test failed - auto-linux-64-opt-mir
http://buildbot.rust-lang.org/builders/auto-linux-64-opt-mir/builds/921

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
#33977 (comment), or mute
the thread
https://github.com/notifications/unsubscribe/AAD95F4amjnnhdjKRVOsJ63Pal3iqZZfks5qHL4DgaJpZM4IqPqp
.

Manishearth added a commit to Manishearth/rust that referenced this pull request Jun 1, 2016
@Manishearth
Rollup merge ofrust-lang#33977- Ms2ger:create-and-enter, r=nrc
42e593a
Take the def_map argument to TyCtxt::create_and_enter out of its RefCell.
@ManishearthManishearth mentioned this pull request Jun 1, 2016
Merged
bors added a commit that referenced this pull request Jun 1, 2016
@bors
Auto merge of#34004- Manishearth:rollup, r=Manishearth
806a553
Rollup of 11 pull requests
- Successful merges: #33385, #33606, #33841, #33892, #33896, #33915, #33921, #33967, #33970, #33973, #33977
- Failed merges:
@bors
bors merged commit 7463c95 into rust-lang:masterJun 1, 2016
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@nrcnrc

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@Ms2ger@rust-highfive@nrc@bors@alexcrichton