From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 102681 invoked by alias); 11 Jun 2016 20:49:29 -0000 Mailing-List: contact binutils-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: binutils-owner@sourceware.org Received: (qmail 102659 invoked by uid 89); 11 Jun 2016 20:49:28 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-0.1 required=5.0 tests=AWL,BAYES_00,SPF_HELO_PASS,SPF_SOFTFAIL autolearn=no version=3.3.2 spammy=2.7.0, Hx-languages-length:1469 X-Spam-User: qpsmtpd, 2 recipients X-HELO: bigwig.baldwin.cx Received: from bigwig.baldwin.cx (HELO bigwig.baldwin.cx) (96.47.65.170) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with (CAMELLIA256-SHA encrypted) ESMTPS; Sat, 11 Jun 2016 20:49:26 +0000 Received: from ralph.baldwin.cx.net (c-73-231-226-104.hsd1.ca.comcast.net [73.231.226.104]) by bigwig.baldwin.cx (Postfix) with ESMTPSA id 7B658B94C; Sat, 11 Jun 2016 16:49:23 -0400 (EDT) From: John Baldwin To: gdb-patches@sourceware.org, binutils@sourceware.org Subject: [PATCH 0/8] Fix various warnings from clang 3.8.0 Date: Sat, 11 Jun 2016 20:49:00 -0000 Message-Id: <1465678115-58170-1-git-send-email-jhb@FreeBSD.org> X-SW-Source: 2016-06/txt/msg00130.txt.bz2 clang triggers a few warnings on binutils and gdb, but I've tried to only submit patches for ones that I think are actual bugs (though in some cases I believe the bugs are harmless in practice). There might be better ways to address some of these warnings. Two that I can think of are: - gdb/score-dep.c might choose to use a 'pc < start' check in the caller of the patched function and throw an error for that case (if it can occurr). - gdb/sh64-tdep.c could explicitly set 'uses_fp' in the cases that it stores a value in the FP register rather than using a check at the end of the function. John Baldwin (8): Use correct enum type for do_elf_stt_common. Change the size field of MSP430_Opcode_Decoded to a plain integer. Initialize 'ra' to zero to avoid uninitialized use. Remove check for negative size. Detect a frameless frame by comparing the FP register to -1. Pass a NULL pointer as the last argument to find_pc_partial_function. Remove unneeded checks on type lengths. Use unsigned integer constant with left shifts. binutils/ChangeLog | 4 ++++ binutils/objcopy.c | 2 +- gdb/ChangeLog | 27 +++++++++++++++++++++++++++ gdb/ada-lang.c | 4 ++-- gdb/rs6000-tdep.c | 1 + gdb/score-tdep.c | 8 +------- gdb/sh64-tdep.c | 2 +- gdb/tui/tui-stack.c | 2 +- gdb/v850-tdep.c | 2 +- include/ChangeLog | 5 +++++ include/opcode/msp430-decode.h | 9 +-------- 11 files changed, 45 insertions(+), 21 deletions(-) -- 2.7.0