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

Issue 1704053002: Documentation change - Remove MIPS from LIBYUV_DISABLE_DSPR2 (Closed)

Created:
4 years, 10 months ago by fbarchard
Modified:
4 years, 10 months ago
CC:
magjed1
Base URL:
https://chromium.googlesource.com/libyuv/libyuv@master
Target Ref:
refs/heads/master
Project:
libyuv
Visibility:
Public.

Description

Documentation change - Remove MIPS from LIBYUV_DISABLE_DSPR2 Make environment variables match ifdefs with one SIMD instruction set word as suffix for functions, ifdefs and environment variables. R=kjellander@chromium.org BUG=libyuv:486 Committed: https://chromium.googlesource.com/libyuv/libyuv/+/c5745d558a1992208631d432c1622839098bb062

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -1 line) Patch
M docs/environment_variables.md View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 9 (3 generated)
fbarchard
4 years, 10 months ago (2016-02-17 18:49:52 UTC) #1
fbarchard1
when attempting to make row function names more consistent, mips_dspr2 was changed to just DSPR2 ...
4 years, 10 months ago (2016-02-17 18:56:49 UTC) #4
kjellander_chromium
I see, lgtm. No need to run trybots for a docs-only change :)
4 years, 10 months ago (2016-02-17 19:02:07 UTC) #5
fbarchard1
On 2016/02/17 19:02:07, kjellander (chromium) wrote: > I see, lgtm. > No need to run ...
4 years, 10 months ago (2016-02-17 19:07:14 UTC) #6
fbarchard
Committed patchset #1 (id:1) manually as c5745d558a1992208631d432c1622839098bb062 (presubmit successful).
4 years, 10 months ago (2016-02-17 19:07:32 UTC) #8
fbarchard1
4 years, 10 months ago (2016-02-17 19:08:21 UTC) #9
Message was sent while issue was closed.
On 2016/02/17 19:07:32, fbarchard wrote:
> Committed patchset #1 (id:1) manually as
> c5745d558a1992208631d432c1622839098bb062 (presubmit successful).

git cl land worked.

Powered by Google App Engine
This is Rietveld 408576698