Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(383)

Issue 1921733003: PPC: [turbofan] Introduce TruncateTaggedToWord32 simplified operator. (Closed)

Created:
4 years, 8 months ago by JaideepBajwa
Modified:
4 years, 7 months ago
CC:
v8-reviews_googlegroups.com
Base URL:
https://chromium.googlesource.com/v8/v8.git@master
Target Ref:
refs/pending/heads/master
Project:
v8
Visibility:
Public.

Description

PPC: [turbofan] Introduce TruncateTaggedToWord32 simplified operator. Port 0231a7efa2b5ce8108a837284e2f63dccd5dcd09 Original commit message: This allows us to get rid of the "push TruncateFloat64ToInt32 into Phi" trick that was used in the MachineOperatorReducer to combine the ChangeTaggedToFloat64 and TruncateFloat64ToInt32 operations. Instead of doing that later, we can just introduce the proper operator during the representation selection directly. Also separate the TruncateFloat64ToInt32 machine operator, which had two different meanings depending on a flag (either JavaScript truncation or C++ style round to zero). Now there's a TruncateFloat64ToWord32 which represents the JavaScript truncation (implemented via TruncateDoubleToI macro + code stub) and the RoundFloat64ToInt32, which implements the C++ round towards zero operation (in the same style as the other WebAssembly driven Round* machine operators). R=bmeurer@chromium.org, joransiu@ca.ibm.com, jyan@ca.ibm.com, michael_dawson@ca.ibm.com, mbrandy@us.ibm.com BUG= LOG=N Committed: https://crrev.com/ff243a95797cfd7b4cf446dff5ca33fc367c984b Cr-Commit-Position: refs/heads/master@{#35771}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+5 lines, -7 lines) Patch
M src/compiler/ppc/instruction-selector-ppc.cc View 1 chunk +5 lines, -7 lines 0 comments Download

Messages

Total messages: 7 (2 generated)
JaideepBajwa
PTAL
4 years, 8 months ago (2016-04-25 15:39:34 UTC) #1
MTBrandyberry
lgtm
4 years, 8 months ago (2016-04-25 15:42:13 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1921733003/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1921733003/1
4 years, 8 months ago (2016-04-25 16:03:58 UTC) #4
commit-bot: I haz the power
Committed patchset #1 (id:1)
4 years, 8 months ago (2016-04-25 16:27:24 UTC) #5
commit-bot: I haz the power
4 years, 8 months ago (2016-04-25 16:29:16 UTC) #7
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/ff243a95797cfd7b4cf446dff5ca33fc367c984b
Cr-Commit-Position: refs/heads/master@{#35771}

Powered by Google App Engine
This is Rietveld 408576698