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

Issue 1111733002: [clang] Use -Wshorten-64-to-32 to enable warnings about 64bit to 32bit truncations. (Closed)

Created:
5 years, 7 months ago by Benedikt Meurer
Modified:
5 years, 7 months ago
Reviewers:
Sven Panne
CC:
v8-dev, titzer, Rodolph Perfetta (ARM)
Base URL:
https://chromium.googlesource.com/v8/v8.git@master
Target Ref:
refs/pending/heads/master
Project:
v8
Visibility:
Public.

Description

[clang] Use -Wshorten-64-to-32 to enable warnings about 64bit to 32bit truncations. Currently only the Win64 bots report this warnings, which adds quite some overhead to the development process. With this flag we also get compiler warnings about implicit 64bit to 32bit truncations when building with clang on Linux/x64 and Mac/x64. R=svenpanne@chromium.org Committed: https://crrev.com/4486c47d9b207e95f15598135262bb43f0c5dc00 Cr-Commit-Position: refs/heads/master@{#28093}

Patch Set 1 #

Patch Set 2 : Fixes #

Patch Set 3 : Fixes #

Patch Set 4 : Fix Win warnings. #

Patch Set 5 : Fix Win warnings. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+163 lines, -491 lines) Patch
M build/standalone.gypi View 1 2 2 chunks +6 lines, -0 lines 0 comments Download
M samples/process.cc View 1 2 3 1 chunk +9 lines, -6 lines 0 comments Download
M samples/shell.cc View 1 2 3 1 chunk +9 lines, -6 lines 0 comments Download
M src/base/platform/platform.h View 1 chunk +6 lines, -4 lines 0 comments Download
M src/base/platform/platform-aix.cc View 1 chunk +0 lines, -49 lines 0 comments Download
M src/base/platform/platform-cygwin.cc View 1 chunk +0 lines, -48 lines 0 comments Download
M src/base/platform/platform-freebsd.cc View 1 chunk +0 lines, -48 lines 0 comments Download
M src/base/platform/platform-linux.cc View 3 chunks +2 lines, -60 lines 0 comments Download
M src/base/platform/platform-macos.cc View 1 chunk +0 lines, -58 lines 0 comments Download
M src/base/platform/platform-openbsd.cc View 1 chunk +0 lines, -48 lines 0 comments Download
M src/base/platform/platform-posix.cc View 3 chunks +62 lines, -3 lines 0 comments Download
M src/base/platform/platform-qnx.cc View 1 chunk +0 lines, -58 lines 0 comments Download
M src/base/platform/platform-solaris.cc View 1 chunk +0 lines, -48 lines 0 comments Download
M src/base/platform/platform-win32.cc View 1 2 3 4 3 chunks +24 lines, -23 lines 0 comments Download
M src/base/platform/time.cc View 1 3 chunks +3 lines, -3 lines 0 comments Download
M src/d8.cc View 1 2 3 1 chunk +9 lines, -5 lines 0 comments Download
M src/d8-posix.cc View 5 chunks +9 lines, -9 lines 0 comments Download
M src/flags.cc View 1 chunk +1 line, -1 line 0 comments Download
M src/gdb-jit.cc View 7 chunks +8 lines, -7 lines 0 comments Download
M src/snapshot/mksnapshot.cc View 1 1 chunk +4 lines, -4 lines 0 comments Download
M src/utils.cc View 1 chunk +1 line, -1 line 0 comments Download
M test/cctest/compiler/call-tester.h View 1 2 2 chunks +6 lines, -0 lines 0 comments Download
M test/cctest/test-mark-compact.cc View 1 chunk +1 line, -1 line 0 comments Download
M tools/shell-utils.h View 2 chunks +3 lines, -1 line 0 comments Download

Messages

Total messages: 9 (1 generated)
Benedikt Meurer
5 years, 7 months ago (2015-04-28 05:38:47 UTC) #1
Benedikt Meurer
Sven: This adds the compiler flag and fixes all invalid 64bit to 32bit truncations detected ...
5 years, 7 months ago (2015-04-28 05:40:13 UTC) #2
Benedikt Meurer
Rodolph: Maybe you can look into enabling this warning for arm64 as well. It gives ...
5 years, 7 months ago (2015-04-28 06:06:44 UTC) #3
Sven Panne
lgtm
5 years, 7 months ago (2015-04-28 06:47:56 UTC) #5
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1111733002/80001
5 years, 7 months ago (2015-04-28 06:48:00 UTC) #6
commit-bot: I haz the power
Committed patchset #5 (id:80001)
5 years, 7 months ago (2015-04-28 06:54:13 UTC) #7
commit-bot: I haz the power
Patchset 5 (id:??) landed as https://crrev.com/4486c47d9b207e95f15598135262bb43f0c5dc00 Cr-Commit-Position: refs/heads/master@{#28093}
5 years, 7 months ago (2015-04-28 06:54:25 UTC) #8
Rodolph Perfetta
5 years, 7 months ago (2015-04-28 10:54:28 UTC) #9
Message was sent while issue was closed.
On 2015/04/28 06:06:44, Benedikt Meurer wrote:
> Rodolph: Maybe you can look into enabling this warning for arm64 as well. It
> gives a bunch of warnings in the arm64/ directory that are related to mixing
int
> and int64_t.

We are looking into it.

Powered by Google App Engine
This is Rietveld 408576698