Skip to content

Commit e9f6b8e

Browse files
committed
Convert a todo! to a FIXME in the trace optimiser.
This scenario can indeed happen. I've just seen it with the Lua tests with the "really long trace" fix that's pending to merge.
1 parent 86074f7 commit e9f6b8e

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

‎ykrt/src/compile/jitc_yk/opt/analyse.rs‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -69,8 +69,8 @@ impl Analyse {
6969
if let (&Operand::Const(_lhs_cidx), &Operand::Const(_rhs_cidx)) =
7070
(&lhs, &rhs)
7171
{
72-
// Can we still hit this case?
73-
todo!();
72+
// FIXME: implement the optimisation.
73+
return op;
7474
}
7575
}
7676
op

0 commit comments

Comments
 (0)