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

Issue 2425623004: cpu_id cleanup. no functional change. (Closed)

Created:
4 years, 2 months ago by fbarchard1
Modified:
4 years, 2 months ago
Target Ref:
refs/heads/master
Project:
libyuv
Visibility:
Public.

Description

cpu_id cleanup. no functional change. remove old comment about initialize to zero. remove ifdef and replace with macro defined to zero. BUG=None TEST=try bots R=kjellander@chromium.org Committed: https://chromium.googlesource.com/libyuv/libyuv/+/e16e3a629f0c1c6ea275fac9798312647056a1ab

Patch Set 1 #

Patch Set 2 : cpu_id cleanup. no functional change. #

Patch Set 3 : cpu_id cleanup. no functional change. #

Patch Set 4 : cpu_id cleanup comments/ifdefs and add denormal test for halffloat. #

Patch Set 5 : remove planar test change #

Unified diffs Side-by-side diffs Delta from patch set Stats (+5 lines, -6 lines) Patch
M source/cpu_id.cc View 1 2 3 6 chunks +5 lines, -6 lines 0 comments Download

Messages

Total messages: 5 (3 generated)
kjellander_chromium
lgtm
4 years, 2 months ago (2016-10-18 19:18:23 UTC) #3
fbarchard1
4 years, 2 months ago (2016-10-18 19:26:07 UTC) #5
Message was sent while issue was closed.
Committed patchset #5 (id:80001) manually as
e16e3a629f0c1c6ea275fac9798312647056a1ab (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698