Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add mir building test directory #101428

Merged
merged 2 commits into from
Oct 31, 2022
Merged

Add mir building test directory #101428

merged 2 commits into from
Oct 31, 2022

Conversation

JakobDegen
Copy link
Contributor

@JakobDegen JakobDegen commented Sep 5, 2022

The first commit renames mir-map.0 mir dumps to built.after dumps. I am happy to drop this commit if someone can explain the origin of the name.

The second commit moves a bunch of mir building tests into their own directory. I did my best to make sure that all of these tests are actually testing mir building, and not just incidentally using built.after

r? @oli-obk

@rustbot rustbot added the T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. label Sep 5, 2022
@rustbot
Copy link
Collaborator

rustbot commented Sep 5, 2022

Some changes occurred to MIR optimizations

cc @rust-lang/wg-mir-opt

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Sep 5, 2022
@oli-obk
Copy link
Contributor

oli-obk commented Sep 6, 2022

I thought we were already dumping right after mir building (although with a weird name)

@JakobDegen
Copy link
Contributor Author

Yeah, PromoteTemps.before. I can just switch to using that if you want, but I figured some clear naming might be nice

@cjgillot
Copy link
Contributor

cjgillot commented Sep 6, 2022

There is the mir_map dump. Should it be removed?

@oli-obk
Copy link
Contributor

oli-obk commented Sep 7, 2022

There is the mir_map dump. Should it be removed?

This is the one I meant.

@oli-obk
Copy link
Contributor

oli-obk commented Sep 16, 2022

@rustbot author

@rustbot rustbot 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-review Status: Awaiting review from the assignee but also interested parties. labels Sep 16, 2022
@bors
Copy link
Collaborator

bors commented Sep 17, 2022

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

@JakobDegen
Copy link
Contributor Author

Gonna wait on #102340 before fixing this

@JakobDegen JakobDegen changed the title Add mir dump for after mir building and use it in tests Add mir building test directory Oct 27, 2022
@JakobDegen
Copy link
Contributor Author

Rebased, made changes, and updated top comment of PR.

@rustbot ready

@rustbot rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. and removed S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. labels Oct 27, 2022
@oli-obk
Copy link
Contributor

oli-obk commented Oct 27, 2022

@bors r+

@bors
Copy link
Collaborator

bors commented Oct 27, 2022

📌 Commit 51b0363 has been approved by oli-obk

It is now in the queue for this repository.

@bors 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 Oct 27, 2022
notriddle added a commit to notriddle/rust that referenced this pull request Oct 30, 2022
Add mir building test directory

The first commit renames `mir-map.0` mir dumps to `built.after` dumps. I am happy to drop this commit if someone can explain the origin of the name.

The second commit moves a bunch of mir building tests into their own directory. I did my best to make sure that all of these tests are actually testing mir building, and not just incidentally using `built.after`

r? `@oli-obk`
bors added a commit to rust-lang-ci/rust that referenced this pull request Oct 31, 2022
Rollup of 8 pull requests

Successful merges:

 - rust-lang#97971 (Enable varargs support for calling conventions other than C or cdecl )
 - rust-lang#101428 (Add mir building test directory)
 - rust-lang#101944 (rustdoc: clean up `#toggle-all-docs`)
 - rust-lang#102101 (check lld version to choose correct option to disable multi-threading in tests)
 - rust-lang#102689 (Add a tier 3 target for the Sony PlayStation 1)
 - rust-lang#103746 (rustdoc: add support for incoherent impls on structs and traits)
 - rust-lang#103758 (Add regression test for reexports in search results)
 - rust-lang#103764 (All verbosity checks in `PrettyPrinter` now go through `PrettyPrinter::should_print_verbose`)

Failed merges:

r? `@ghost`
`@rustbot` modify labels: rollup
@bors bors merged commit 7e62406 into rust-lang:master Oct 31, 2022
@rustbot rustbot added this to the 1.67.0 milestone Oct 31, 2022
@JakobDegen JakobDegen deleted the build-tests branch November 1, 2022 00:17
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants