Skip to content

Commit

Permalink
Fix review comments
Browse files Browse the repository at this point in the history
  • Loading branch information
bjorn3 committed Oct 9, 2023
1 parent 7f5af72 commit f0b5820
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 5 deletions.
2 changes: 1 addition & 1 deletion compiler/rustc_codegen_cranelift/src/driver/aot.rs
Original file line number Diff line number Diff line change
Expand Up @@ -412,7 +412,7 @@ pub(crate) fn run_aot(
)
.0
}
CguReuse::PreLto => unreachable!(),
CguReuse::PreLto => unreachable!("LTO not yet supported"),
CguReuse::PostLto => {
concurrency_limiter.job_already_done();
OngoingModuleCodegen::Sync(reuse_workproduct_for_cgu(tcx, cgu))
Expand Down
8 changes: 4 additions & 4 deletions compiler/rustc_codegen_ssa/src/assert_module_sources.rs
Original file line number Diff line number Diff line change
Expand Up @@ -227,14 +227,14 @@ pub struct CguReuseTracker {
}

impl CguReuseTracker {
pub fn new() -> CguReuseTracker {
fn new() -> CguReuseTracker {
let data =
TrackerData { actual_reuse: Default::default(), expected_reuse: Default::default() };

CguReuseTracker { data: Some(data) }
}

pub fn new_disabled() -> CguReuseTracker {
fn new_disabled() -> CguReuseTracker {
CguReuseTracker { data: None }
}

Expand All @@ -247,7 +247,7 @@ impl CguReuseTracker {
}
}

pub fn set_expectation(
fn set_expectation(
&mut self,
cgu_name: Symbol,
cgu_user_name: &str,
Expand All @@ -265,7 +265,7 @@ impl CguReuseTracker {
}
}

pub fn check_expected_reuse(&self, sess: &Session) {
fn check_expected_reuse(&self, sess: &Session) {
if let Some(ref data) = self.data {
for (cgu_name, &(ref cgu_user_name, ref error_span, expected_reuse, comparison_kind)) in
&data.expected_reuse
Expand Down

0 comments on commit f0b5820

Please sign in to comment.