Skip to content

Commit

Permalink
fix: correct ICE panic messages in brillig convert_black_box_call (#…
Browse files Browse the repository at this point in the history
…4761)

# Description

## Problem\*

Related to #4687 

## Summary\*

The error messages here were clearly copy-pasted without any updates so
this PR fixes these to match the actual black box function they're for.

## Additional Context



## Documentation\*

Check one:
- [x] No documentation needed.
- [ ] Documentation included in this PR.
- [ ] **[For Experimental Features]** Documentation to be submitted in a
separate PR.

# PR Checklist\*

- [x] I have tested the changes locally.
- [x] I have formatted the changes with [Prettier](https://prettier.io/)
and/or `cargo fmt` on default settings.
  • Loading branch information
TomAFrench authored Apr 9, 2024
1 parent b8dd770 commit f3eee6c
Showing 1 changed file with 6 additions and 6 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -246,7 +246,7 @@ pub(crate) fn convert_black_box_call(
});
} else {
unreachable!(
"ICE: EmbeddedCurveAdd expects two register arguments and one array result"
"ICE: BigIntAdd expects two register arguments and one result register"
)
}
}
Expand All @@ -263,7 +263,7 @@ pub(crate) fn convert_black_box_call(
});
} else {
unreachable!(
"ICE: EmbeddedCurveAdd expects two register arguments and one array result"
"ICE: BigIntSub expects two register arguments and one result register"
)
}
}
Expand All @@ -280,7 +280,7 @@ pub(crate) fn convert_black_box_call(
});
} else {
unreachable!(
"ICE: EmbeddedCurveAdd expects two register arguments and one array result"
"ICE: BigIntMul expects two register arguments and one result register"
)
}
}
Expand All @@ -297,7 +297,7 @@ pub(crate) fn convert_black_box_call(
});
} else {
unreachable!(
"ICE: EmbeddedCurveAdd expects two register arguments and one array result"
"ICE: BigIntDiv expects two register arguments and one result register"
)
}
}
Expand All @@ -314,7 +314,7 @@ pub(crate) fn convert_black_box_call(
});
} else {
unreachable!(
"ICE: EmbeddedCurveAdd expects two register arguments and one array result"
"ICE: BigIntFromLeBytes expects two register arguments and one result register"
)
}
}
Expand All @@ -330,7 +330,7 @@ pub(crate) fn convert_black_box_call(
});
} else {
unreachable!(
"ICE: EmbeddedCurveAdd expects two register arguments and one array result"
"ICE: BigIntToLeBytes expects one register argument and one array result"
)
}
}
Expand Down

0 comments on commit f3eee6c

Please sign in to comment.