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

Issue 519034: Use no_strict_aliasing=1 by default. (Closed)

Created:
10 years, 11 months ago by Craig
Modified:
9 years, 7 months ago
Reviewers:
Evan Martin
CC:
chromium-reviews_googlegroups.com
Visibility:
Public.

Description

Use no_strict_aliasing=1 by default. This will turn on -fno-strict-aliasing. There are regular aliasing problems in the codebase and at least one of these was not caught by -Wall -Werror so defaulting to -fno-strict-aliasing seems like a good idea especially given the increased popularity of gcc 4.4 with it's use in karmic, fedora and in chromeos chroot builds et. al. The long term plan is to have some test infrastructure in place that will test 4.4 binaries and to fix the aliasing issues. BUG=28749 Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=36151

Patch Set 1 #

Total comments: 1

Patch Set 2 : do it for all compilers #

Unified diffs Side-by-side diffs Delta from patch set Stats (+7 lines, -1 line) Patch
M build/common.gypi View 1 1 chunk +7 lines, -1 line 0 comments Download

Messages

Total messages: 3 (0 generated)
Evan Martin
http://codereview.chromium.org/519034/diff/1/2 File build/common.gypi (right): http://codereview.chromium.org/519034/diff/1/2#newcode585 build/common.gypi:585: {'no_strict_aliasing%': 0,},], Maybe we should just default the variable ...
10 years, 11 months ago (2010-01-13 18:01:23 UTC) #1
Craig
On 2010/01/13 18:01:23, Evan Martin wrote: > http://codereview.chromium.org/519034/diff/1/2 > File build/common.gypi (right): > > http://codereview.chromium.org/519034/diff/1/2#newcode585 ...
10 years, 11 months ago (2010-01-13 18:27:16 UTC) #2
Evan Martin
10 years, 11 months ago (2010-01-13 18:29:57 UTC) #3
LGTM

Powered by Google App Engine
This is Rietveld 408576698