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

Issue 1531303004: Fix regression in SSE instruction opcodes (Closed)

Created:
5 years ago by sehr
Modified:
5 years ago
CC:
native-client-reviews_googlegroups.com
Base URL:
https://chromium.googlesource.com/native_client/pnacl-subzero.git@master
Target Ref:
refs/heads/master
Visibility:
Public.

Description

Patch Set 1 #

Total comments: 2
Unified diffs Side-by-side diffs Delta from patch set Stats (+8 lines, -8 lines) Patch
M src/IceInstX86Base.h View 3 chunks +8 lines, -8 lines 2 comments Download

Messages

Total messages: 8 (3 generated)
sehr
Sorry for the regression. PTAL.
5 years ago (2015-12-18 00:48:24 UTC) #3
Jim Stichnoth
lgtm
5 years ago (2015-12-18 00:49:54 UTC) #4
John
lgtm https://codereview.chromium.org/1531303004/diff/1/src/IceInstX86Base.h File src/IceInstX86Base.h (left): https://codereview.chromium.org/1531303004/diff/1/src/IceInstX86Base.h#oldcode3077 src/IceInstX86Base.h:3077: template <> const char *InstX86Addps<Machine>::Base::Opcode = "addps"; \ ...
5 years ago (2015-12-18 00:50:03 UTC) #5
sehr
Thx. https://codereview.chromium.org/1531303004/diff/1/src/IceInstX86Base.h File src/IceInstX86Base.h (left): https://codereview.chromium.org/1531303004/diff/1/src/IceInstX86Base.h#oldcode3077 src/IceInstX86Base.h:3077: template <> const char *InstX86Addps<Machine>::Base::Opcode = "addps"; \ ...
5 years ago (2015-12-18 00:52:12 UTC) #6
sehr
5 years ago (2015-12-18 00:53:17 UTC) #8
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
19413a5663260165201a0c90a4354c54bf2c5e77 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698