New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 646279 link

Starred by 2 users

Issue metadata

Status: Duplicate
Merged: issue libyuv:637
Owner:
Last visit > 30 days ago
Closed: Sep 2016
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Feature

Blocked on:
issue libyuv:637



Sign in to add a comment

Enable neon on iOS 64

Project Member Reported by ehmaldonado@chromium.org, Sep 13 2016

Issue description

Neon is currently disabled for iOS 64, which is crippling performance.

https://cs.chromium.org/chromium/src/third_party/libyuv/libyuv_test.gyp?rcl=0&l=55
 
This should be reported as a libyuv issue here:

https://bugs.chromium.org/p/libyuv/issues/list

Mergedinto: libyuv:637
Status: Duplicate (was: Assigned)
Blockedon: libyuv:637
Project Member

Comment 5 by bugdroid1@chromium.org, Sep 30 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/1b959ada6e8cb860d4fe25a0aa6fb0a16899b1aa

commit 1b959ada6e8cb860d4fe25a0aa6fb0a16899b1aa
Author: fbarchard <fbarchard@google.com>
Date: Fri Sep 30 16:54:39 2016

libyuv: bt.709 coefficients update, halffloat support, mips msa

bt.709 coefficients in chrome media at video range, while libyuv
originally implemented full range.  This changes libyuv to match
media's video range coefficients for H420ToARGB

HalfFloatPlane function added for converting shorts to half floats
for gpu texture support.  Uses intel instruction vcvtps2ph and
provides a C fallback for other processors.

Mips MSA initial SIMD support for new instruction set. MSA provides
16 byte wide registers, vs DSPR2 was 4 bytes.  Performance is
approximately 12x vs C versions of I420Mirror and I420ToYUY2.

Change log:
https://chromium.googlesource.com/libyuv/libyuv/+log/de944ed8..7fc932dd
https://chromium.googlesource.com/libyuv/libyuv/+/de944ed8..7fc932dd

BUG= libyuv:560 ,  libyuv:639 ,  chromium:445071 , libyuv:635 ,  libyuv:636 ,  libyuv:637 ,  chromium:646279 ,  libyuv:634 
TEST=try bots and new unittests added for halffloat.
R=hubbe@chromium.org

Review-Url: https://codereview.chromium.org/2378773008
Cr-Commit-Position: refs/heads/master@{#422132}

[modify] https://crrev.com/1b959ada6e8cb860d4fe25a0aa6fb0a16899b1aa/DEPS

Project Member

Comment 6 by bugdroid1@chromium.org, Sep 30 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/1b959ada6e8cb860d4fe25a0aa6fb0a16899b1aa

commit 1b959ada6e8cb860d4fe25a0aa6fb0a16899b1aa
Author: fbarchard <fbarchard@google.com>
Date: Fri Sep 30 16:54:39 2016

libyuv: bt.709 coefficients update, halffloat support, mips msa

bt.709 coefficients in chrome media at video range, while libyuv
originally implemented full range.  This changes libyuv to match
media's video range coefficients for H420ToARGB

HalfFloatPlane function added for converting shorts to half floats
for gpu texture support.  Uses intel instruction vcvtps2ph and
provides a C fallback for other processors.

Mips MSA initial SIMD support for new instruction set. MSA provides
16 byte wide registers, vs DSPR2 was 4 bytes.  Performance is
approximately 12x vs C versions of I420Mirror and I420ToYUY2.

Change log:
https://chromium.googlesource.com/libyuv/libyuv/+log/de944ed8..7fc932dd
https://chromium.googlesource.com/libyuv/libyuv/+/de944ed8..7fc932dd

BUG= libyuv:560 ,  libyuv:639 ,  chromium:445071 , libyuv:635 ,  libyuv:636 ,  libyuv:637 ,  chromium:646279 ,  libyuv:634 
TEST=try bots and new unittests added for halffloat.
R=hubbe@chromium.org

Review-Url: https://codereview.chromium.org/2378773008
Cr-Commit-Position: refs/heads/master@{#422132}

[modify] https://crrev.com/1b959ada6e8cb860d4fe25a0aa6fb0a16899b1aa/DEPS

Sign in to add a comment