Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Fix bigint side effects for >>>, /, %
Summary: BigInt differs from numbers - several bigint operations can throw an exception, specifically: - >>> is not allowed at all and throws - / and % can result in division by zero Improve the side effect model and add a test. Reported in #1212 Reviewed By: avp Differential Revision: D52013220 fbshipit-source-id: 34606cb7d53a3db3ea49dd026661b7fd340583e5
- Loading branch information
1 parent
40748fa
commit 39d60e6
Showing
2 changed files
with
45 additions
and
5 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters