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

Issue 1922183002: S390: [turbofan] Introduce TruncateTaggedToWord32 simplified operator. (Closed)

Created:
4 years, 8 months ago by john.yan
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

S390: [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, bjaideep@ca.ibm.com, michael_dawson@ca.ibm.com, mbrandy@us.ibm.com BUG= LOG=N Committed: https://crrev.com/1decc266bad6457f7da2a8c67609a757fb85f3a3 Cr-Commit-Position: refs/heads/master@{#35802}

Patch Set 1 #

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

Messages

Total messages: 11 (4 generated)
john.yan
PTAL
4 years, 8 months ago (2016-04-26 17:46:07 UTC) #1
commit-bot: I haz the power
Dry run: CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1922183002/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1922183002/1
4 years, 8 months ago (2016-04-26 18:33:07 UTC) #3
commit-bot: I haz the power
Dry run: This issue passed the CQ dry run.
4 years, 8 months ago (2016-04-26 19:05:26 UTC) #5
JoranSiu
On 2016/04/26 19:05:26, commit-bot: I haz the power wrote: > Dry run: This issue passed ...
4 years, 8 months ago (2016-04-26 20:36:51 UTC) #6
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1922183002/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1922183002/1
4 years, 8 months ago (2016-04-26 21:35:06 UTC) #8
commit-bot: I haz the power
Committed patchset #1 (id:1)
4 years, 8 months ago (2016-04-26 21:38:09 UTC) #9
commit-bot: I haz the power
4 years, 8 months ago (2016-04-26 21:38:37 UTC) #11
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/1decc266bad6457f7da2a8c67609a757fb85f3a3
Cr-Commit-Position: refs/heads/master@{#35802}

Powered by Google App Engine
This is Rietveld 408576698