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

Issue 20162004: Update perf expectations for PNaCl after 3.3 merge, etc. (Closed)

Created:
7 years, 5 months ago by jvoung (off chromium)
Modified:
7 years, 5 months ago
Reviewers:
Jim Stichnoth, JF
CC:
native-client-reviews_googlegroups.com
Visibility:
Public.

Description

Update perf expectations for PNaCl after 3.3 merge, etc. * compile time is faster (we run IR verification 1x instead of 2x) * some upstream improvements to compiletime (stats off w/ asserts off) * ARM gzip runs mysteriously faster in -O0 and -O2 * X86-32 vortex runs mysteriously slower at -O2 * update pexe transfer size Eon for x86-64 as well (same as x86-32) that was updated last time, but x86-64 was left out. Will investigate vortex more, but for now make the tree stop closing. BUG=https://code.google.com/p/nativeclient/issues/detail?id=3593 R=stichnot@chromium.org Committed: https://src.chromium.org/viewvc/native_client?view=rev&revision=11851

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+68 lines, -68 lines) Patch
M tools/nacl_perf_expectations/nacl_perf_expectations.json View 5 chunks +68 lines, -68 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
jvoung (off chromium)
7 years, 5 months ago (2013-07-24 22:44:03 UTC) #1
Jim Stichnoth
lgtm
7 years, 5 months ago (2013-07-24 22:45:49 UTC) #2
jvoung (off chromium)
7 years, 5 months ago (2013-07-24 22:56:15 UTC) #3
Message was sent while issue was closed.
Committed patchset #1 manually as r11851 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698