Skip to content

Rollup of 8 pull requests #115928

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

Closed
wants to merge 30 commits into from
Closed
Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
30 commits
Select commit Hold shift + click to select a range
86d32ef
issue has since been fixed
tshepang Sep 5, 2023
7db123d
Add myself to the mailmap
tgross35 Sep 10, 2023
074fb2c
Store target triple in environment
Kobzol Sep 12, 2023
95500f4
Make executable extension platform, rather than environment dependent
Kobzol Sep 12, 2023
f17047b
Refactor Environment
Kobzol Sep 12, 2023
a2ed508
Fix `reset_directory` function
Kobzol Sep 12, 2023
11f9283
Add a Local environment to `opt-dist`
Kobzol Sep 12, 2023
2962528
Make AIX known by bootstrap
Sep 13, 2023
fc17e09
Add comment to elaborate
Sep 13, 2023
5049a71
Adjust comment
Sep 13, 2023
122d1cc
Adjust comment
Sep 13, 2023
6c718b5
Refactor rustc-perf building
Kobzol Sep 13, 2023
f13b545
Resolve clippy warnings
Kobzol Sep 13, 2023
be9d7e0
`GoalCandidate` to `Probe`
lcnr Sep 14, 2023
a3f9530
order `added_goals_evaluation` and `nested_probes`
lcnr Sep 14, 2023
1b141b6
inspect: explicitly store added goals
lcnr Sep 14, 2023
0cb800e
differentiate root and nested goals
lcnr Sep 14, 2023
5b882ac
Bump to supported Ubuntu
Mark-Simulacrum Sep 12, 2023
abd265e
Move to older, mirrored redox install
Mark-Simulacrum Sep 13, 2023
c70ee68
Add me as on vacation
jackh726 Sep 17, 2023
3b817b2
nop_lift macros: ensure that we are using the right interner
RalfJung Sep 17, 2023
a716c96
address review comment
tshepang Sep 18, 2023
23d4457
Rollup merge of #115558 - tshepang:patch-4, r=Mark-Simulacrum
matthiaskrgr Sep 18, 2023
2239734
Rollup merge of #115724 - tgross35:mailmap, r=Mark-Simulacrum
matthiaskrgr Sep 18, 2023
afdfce3
Rollup merge of #115795 - Kobzol:opt-dist-custom, r=Mark-Simulacrum
matthiaskrgr Sep 18, 2023
d958bf3
Rollup merge of #115811 - bzEq:make-aix-known, r=Mark-Simulacrum
matthiaskrgr Sep 18, 2023
2d170f1
Rollup merge of #115838 - lcnr:added-goals, r=compiler-errors
matthiaskrgr Sep 18, 2023
c86427f
Rollup merge of #115902 - Mark-Simulacrum:bump-ci, r=albertlarsan68
matthiaskrgr Sep 18, 2023
e7d7a4d
Rollup merge of #115907 - RalfJung:interner-check, r=compiler-errors
matthiaskrgr Sep 18, 2023
8851323
Rollup merge of #115916 - jackh726:vacation, r=jackh726
matthiaskrgr Sep 18, 2023
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Resolve clippy warnings
  • Loading branch information
Kobzol committed Sep 13, 2023
commit f13b54546bb7aea9beee7d3f9178882a7f10f45c
6 changes: 3 additions & 3 deletions src/tools/opt-dist/src/tests.rs
Original file line number Diff line number Diff line change
Expand Up @@ -33,8 +33,8 @@ pub fn run_tests(env: &Environment) -> anyhow::Result<()> {

// We need to manually copy libstd to the extracted rustc sysroot
copy_directory(
&libstd_dir.join("lib").join("rustlib").join(&host_triple).join("lib"),
&rustc_dir.join("lib").join("rustlib").join(&host_triple).join("lib"),
&libstd_dir.join("lib").join("rustlib").join(host_triple).join("lib"),
&rustc_dir.join("lib").join("rustlib").join(host_triple).join("lib"),
)?;

// Extract sources - they aren't in the `rustc-nightly-{host}` tarball, so we need to manually copy libstd
Expand Down Expand Up @@ -109,6 +109,6 @@ fn find_dist_version(directory: &Utf8Path) -> anyhow::Result<String> {
.unwrap()
.to_string();
let (version, _) =
archive.strip_prefix("reproducible-artifacts-").unwrap().split_once("-").unwrap();
archive.strip_prefix("reproducible-artifacts-").unwrap().split_once('-').unwrap();
Ok(version.to_string())
}
2 changes: 1 addition & 1 deletion src/tools/opt-dist/src/training.rs
Original file line number Diff line number Diff line change
Expand Up @@ -192,7 +192,7 @@ pub fn gather_llvm_bolt_profiles(env: &Environment) -> anyhow::Result<LlvmBoltPr
log::info!("Merging LLVM BOLT profiles to {merged_profile}");

let profiles: Vec<_> =
glob::glob(&format!("{profile_root}*"))?.into_iter().collect::<Result<Vec<_>, _>>()?;
glob::glob(&format!("{profile_root}*"))?.collect::<Result<Vec<_>, _>>()?;

let mut merge_args = vec!["merge-fdata"];
merge_args.extend(profiles.iter().map(|p| p.to_str().unwrap()));
Expand Down
6 changes: 2 additions & 4 deletions src/tools/opt-dist/src/utils/io.rs
Original file line number Diff line number Diff line change
Expand Up @@ -63,7 +63,6 @@ pub fn get_files_from_dir(
let path = format!("{dir}/*{}", suffix.unwrap_or(""));

Ok(glob::glob(&path)?
.into_iter()
.map(|p| p.map(|p| Utf8PathBuf::from_path_buf(p).unwrap()))
.collect::<Result<Vec<_>, _>>()?)
}
Expand All @@ -74,9 +73,8 @@ pub fn find_file_in_dir(
prefix: &str,
suffix: &str,
) -> anyhow::Result<Utf8PathBuf> {
let files = glob::glob(&format!("{directory}/{prefix}*{suffix}"))?
.into_iter()
.collect::<Result<Vec<_>, _>>()?;
let files =
glob::glob(&format!("{directory}/{prefix}*{suffix}"))?.collect::<Result<Vec<_>, _>>()?;
match files.len() {
0 => Err(anyhow::anyhow!("No file with prefix {prefix} found in {directory}")),
1 => Ok(Utf8PathBuf::from_path_buf(files[0].clone()).unwrap()),
Expand Down