@@ -133,7 +133,7 @@ pub fn inline(sess: &Session, module: &mut Module) -> super::Result<()> {
133133 . map ( Ok )
134134 . collect ( ) ;
135135
136- // Inline functions in post-order (aka inside-out aka bottom-out ) - that is,
136+ // Inline functions in post-order (aka inside-out aka bottom-up ) - that is,
137137 // callees are processed before their callers, to avoid duplicating work.
138138 for func_idx in call_graph. post_order ( ) {
139139 let mut function = mem:: replace ( & mut functions[ func_idx] , Err ( FuncIsBeingInlined ) ) . unwrap ( ) ;
@@ -411,7 +411,7 @@ fn should_inline(
411411 }
412412
413413 // If the call isn't passing a legal pointer argument (a "memory object",
414- // i.e. an `OpVariable` or one of the caller's `OpFunctionParameters ),
414+ // i.e. an `OpVariable` or one of the caller's `OpFunctionParameter`s ),
415415 // then inlining is required to have a chance at producing legal SPIR-V.
416416 //
417417 // FIXME(eddyb) rewriting away the pointer could be another alternative.
@@ -826,7 +826,7 @@ impl Inliner<'_, '_> {
826826 }
827827
828828 // `vars_and_debuginfo_range.end` indicates where `OpVariable`s
829- // end and other instructions start (module debuginfo), but to
829+ // end and other instructions start (modulo debuginfo), but to
830830 // split the block in two, both sides of the "cut" need "repair":
831831 // - the variables are missing "inlined call frames" pops, that
832832 // may happen later in the block, and have to be synthesized
0 commit comments