Bug 196668
| Summary: | unaryArithShouldSpeculateInt52 and binaryArithShouldSpeculateInt52 are suspect | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Saam Barati <saam> |
| Component: | JavaScriptCore | Assignee: | Saam Barati <saam> |
| Status: | NEW | ||
| Severity: | Normal | CC: | benjamin, fpizlo, ggaren, gskachkov, guijemont, keith_miller, mark.lam, msaboff, rmorisset, ticaiolima, tzagallo, ysuzuki |
| Priority: | P2 | ||
| Version: | WebKit Nightly Build | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
| Bug Depends on: | 196639 | ||
| Bug Blocks: | |||
Saam Barati
E.g, they only really turn into int52 ops when the input is an int52. But like Add, perhaps we should consider converting to int52 when the input is AnyIntAsDouble.
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Saam Barati
(In reply to Saam Barati from comment #0)
> E.g, they only really turn into int52 ops when the input is an int52. But
> like Add, perhaps we should consider converting to int52 when the input is
> AnyIntAsDouble.
I'm not really sure why Add is more special than Mul here.