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

Issue 396603002: disable inline ARM saturated arithmetic assembly for debug builds (Closed)

Created:
6 years, 5 months ago by Mostyn Bramley-Moore
Modified:
6 years, 5 months ago
Reviewers:
picksi, eae, picksi1, eseidel
CC:
blink-reviews, blink-reviews-wtf_chromium.org, Mikhail, abarth-chromium
Base URL:
https://chromium.googlesource.com/chromium/blink.git@master
Project:
blink
Visibility:
Public.

Description

disable inline ARM saturated arithmetic assembly for debug builds When building third_party/WebKit/Source/wtf/asm/SaturatedArithmeticARM.h in debug builds, we get this error: cc1plus: warnings being treated as errors SaturatedArithmeticARM.h: In function 'int saturatedSet(int, int)': SaturatedArithmeticARM.h:70:43: error: asm operand 2 probably doesn't match constraints SaturatedArithmeticARM.h:70:43: error: asm operand 3 probably doesn't match constraints By switching -O0 to -O1 or -O2 in the compile command, the compile succeeds. So let's disable the inline ARM assembly unless the build is optimized- gcc defines _OPTIMIZE_=1 in this case. I think this is a reasonable workaround until we can find a "proper" solution. BUG=325746 Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=178209

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -1 line) Patch
M Source/wtf/SaturatedArithmetic.h View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 11 (0 generated)
Mostyn Bramley-Moore
@picksi, eae: PTAL at this tiny fixup for https://codereview.chromium.org/346913004 - I'm not sure if it's ...
6 years, 5 months ago (2014-07-15 08:28:49 UTC) #1
picksi
lgtm. Thanks for fixing this. I have a heavy handed solution that builds OK for ...
6 years, 5 months ago (2014-07-15 09:22:51 UTC) #2
eseidel
lgtm
6 years, 5 months ago (2014-07-15 15:35:44 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/mostynb@opera.com/396603002/1
6 years, 5 months ago (2014-07-15 15:36:21 UTC) #4
commit-bot: I haz the power
FYI, CQ is re-trying this CL (attempt #1). The failing builders are: linux_blink_dbg on tryserver.blink ...
6 years, 5 months ago (2014-07-15 16:47:11 UTC) #5
commit-bot: I haz the power
The CQ bit was unchecked by commit-bot@chromium.org
6 years, 5 months ago (2014-07-15 17:53:35 UTC) #6
commit-bot: I haz the power
Try jobs failed on following builders: linux_blink_dbg on tryserver.blink (http://build.chromium.org/p/tryserver.blink/builders/linux_blink_dbg/builds/16597)
6 years, 5 months ago (2014-07-15 17:53:36 UTC) #7
Mostyn Bramley-Moore
The CQ bit was checked by mostynb@opera.com
6 years, 5 months ago (2014-07-15 20:01:59 UTC) #8
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/mostynb@opera.com/396603002/1
6 years, 5 months ago (2014-07-15 20:03:25 UTC) #9
commit-bot: I haz the power
FYI, CQ is re-trying this CL (attempt #1). The failing builders are: linux_blink_dbg on tryserver.blink ...
6 years, 5 months ago (2014-07-15 21:11:16 UTC) #10
commit-bot: I haz the power
6 years, 5 months ago (2014-07-15 22:16:02 UTC) #11
Message was sent while issue was closed.
Change committed as 178209

Powered by Google App Engine
This is Rietveld 408576698