Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(148)

Issue 245180043: [libjpeg_turbo] Fix ARM NEON build (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
9 years ago by noel chromium
Modified:
9 years ago
Reviewers:
fdegans
Base URL:
http://src.chromium.org/svn/trunk/deps/third_party/libjpeg_turbo
Visibility:
Public.

Description

[libjpeg_turbo] Fix ARM NEON build On Android, we produce an ARM build with optional NEON code paths. Fix compilation issues when producing a NEON-only builds on ARM - in such builds NEON is not optional. Patch prepared by fdegans@chromium.org BUG=451035

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+26 lines, -2 lines) Patch
M README.chromium View 1 chunk +2 lines, -0 lines 0 comments Download
M google.patch View 1 chunk +22 lines, -0 lines 0 comments Download
M simd/jsimd_arm.c View 2 chunks +2 lines, -2 lines 0 comments Download

Messages

Total messages: 4
noel chromium
Does this look good to you?
9 years ago (2015-06-09 14:27:26 UTC) #1
fdegans
Looks the same as my patch so lgtm thanks!
9 years ago (2015-06-09 14:29:17 UTC) #2
noel chromium
On 2015/06/09 14:29:17, fdegans wrote: > Looks the same as my patch so lgtm thanks! ...
9 years ago (2015-06-09 14:29:59 UTC) #3
noel chromium
9 years ago (2015-06-09 14:37:47 UTC) #4

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b