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

Issue 2066933002: android_full_debug x86 fix - use +rm for width count (Closed)

Created:
4 years, 6 months ago by fbarchard1
Modified:
4 years, 6 months ago
Reviewers:
Diony Rosa
Base URL:
https://chromium.googlesource.com/libyuv/libyuv@master
Target Ref:
refs/heads/master
Project:
libyuv
Visibility:
Public.

Description

android_full_debug x86 fix - use +rm for width count Work around for android full debug build runnign out of registers. 5 functions were running out of registers causing the compiler error error: 'asm' operand has impossible constraints These functions mostly have 4 pointers, a counter (width) and a tempory eax register. With fpic and debug using stackframes, 2 registers are unavailable. So a total of 8 registers are used. Although fpic and stack frame dont apply to assembly, the compiler reserves 2 registers. The optimized version builds, so its likely freeing up the registers once it knows they are not used. These functions used to build, so compile options and/or compiler may have updated.. likely fpic was turned on. An attribute can be done to disable each, and will avoid using the 2 GPR registers, but they are still reserved and unavailable in debug builds on current compilers (gcc 4.9 and clang 3.8). R=dhrosa@google.com BUG=libyuv:602 Committed: https://chromium.googlesource.com/libyuv/libyuv/+/fd3e676e91b30439dd8515fc760093f90d856bb7

Patch Set 1 #

Total comments: 1

Patch Set 2 : version fixed #

Unified diffs Side-by-side diffs Delta from patch set Stats (+8 lines, -8 lines) Patch
M README.chromium View 1 1 chunk +1 line, -1 line 0 comments Download
M include/libyuv/version.h View 1 1 chunk +1 line, -1 line 0 comments Download
M source/row_gcc.cc View 4 chunks +6 lines, -6 lines 0 comments Download

Messages

Total messages: 6 (1 generated)
fbarchard1
4 years, 6 months ago (2016-06-14 18:15:07 UTC) #1
Diony Rosa
https://codereview.chromium.org/2066933002/diff/1/README.chromium File README.chromium (right): https://codereview.chromium.org/2066933002/diff/1/README.chromium#newcode3 README.chromium:3: Version: 1598 Why did the version number jump by ...
4 years, 6 months ago (2016-06-14 18:18:13 UTC) #2
fbarchard1
On 2016/06/14 18:18:13, Diony Rosa wrote: > https://codereview.chromium.org/2066933002/diff/1/README.chromium > File README.chromium (right): > > https://codereview.chromium.org/2066933002/diff/1/README.chromium#newcode3 ...
4 years, 6 months ago (2016-06-14 19:05:19 UTC) #3
Diony Rosa
lgtm
4 years, 6 months ago (2016-06-14 20:14:28 UTC) #4
fbarchard1
4 years, 6 months ago (2016-06-14 22:25:32 UTC) #6
Message was sent while issue was closed.
Committed patchset #2 (id:20001) manually as
fd3e676e91b30439dd8515fc760093f90d856bb7 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698