Description[turbofan] Quickfix for fround built-in lowering.
We have trouble with Math.fround(MEM[...]). Since we now properly type
LoadBuffer (it can produce undefined), lowering of fround has stopped
triggering (as it requires Number type). This CL is a quick fix for this issue
- we simply trigger the lowering for NumberOrUndefined and let representation
selection/truncation analysis deal with this.
Ultimately, we would want to insert some 'simplified' ToNumber conversion
that would be optimized as much as possible during representation
selection.
BUG=chromium:603802
LOG=n
R=bmeurer@chromium.org
Committed: https://crrev.com/71453f1c0f5e402407f6bf5d7c35f52c75cd7eeb
Cr-Commit-Position: refs/heads/master@{#35513}
Patch Set 1 #
Messages
Total messages: 7 (2 generated)
|