Skip to content

Commit

Permalink
Update docs and visibilities of const to op methods
Browse files Browse the repository at this point in the history
  • Loading branch information
oli-obk committed Feb 16, 2019
1 parent 4fdeb2d commit 4b08533
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions src/librustc_mir/interpret/operand.rs
Expand Up @@ -561,7 +561,7 @@ impl<'a, 'mir, 'tcx, M: Machine<'a, 'mir, 'tcx>> EvalContext<'a, 'mir, 'tcx, M>
.collect()
}

// Used when Miri runs into a constant, and by CTFE.
// Used when Miri runs into a constant, and by const propagation.
pub fn eval_lazy_const_to_op(

This comment has been minimized.

Copy link
@RalfJung

RalfJung Feb 16, 2019

Member

So this cannot become crate?

&self,
val: ty::LazyConst<'tcx>,
Expand All @@ -580,8 +580,8 @@ impl<'a, 'mir, 'tcx, M: Machine<'a, 'mir, 'tcx>> EvalContext<'a, 'mir, 'tcx, M>
}
}

// Used when Miri runs into a constant, and by CTFE.
pub fn const_to_op(
// Used when the miri-engine runs into a constant.
crate fn const_to_op(

This comment has been minimized.

Copy link
@RalfJung

RalfJung Feb 16, 2019

Member

By "CTFE" I meant const_eval -- which is outside the miri engine. That explains why this is not private.

&self,
val: ty::Const<'tcx>,
layout: Option<TyLayout<'tcx>>,
Expand Down

0 comments on commit 4b08533

Please sign in to comment.