From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 529 invoked by alias); 25 Nov 2007 22:12:45 -0000 Received: (qmail 521 invoked by uid 22791); 25 Nov 2007 22:12:44 -0000 X-Spam-Check-By: sourceware.org Received: from pool-70-20-17-24.bstnma.fios.verizon.net (HELO ednor.cgf.cx) (70.20.17.24) by sourceware.org (qpsmtpd/0.31) with ESMTP; Sun, 25 Nov 2007 22:12:40 +0000 Received: by ednor.cgf.cx (Postfix, from userid 201) id BE5392B352; Sun, 25 Nov 2007 17:12:38 -0500 (EST) Date: Sun, 25 Nov 2007 22:12:00 -0000 From: Christopher Faylor To: gdb-patches@sourceware.org Subject: Re: [RFC/RFA] Allow cygwin native to compile with --enable-64-bit-bfd Message-ID: <20071125221238.GB10356@ednor.casa.cgf.cx> Mail-Followup-To: gdb-patches@sourceware.org References: <000001c82dad$e507e0b0$af17a210$@u-strasbg.fr> <20071124210708.GE4928@ednor.casa.cgf.cx> <20071124224727.GA13580@caradoc.them.org> <20071125173207.GA7689@ednor.casa.cgf.cx> <20071125190823.GA11606@caradoc.them.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20071125190823.GA11606@caradoc.them.org> User-Agent: Mutt/1.5.16 (2007-06-09) Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org X-SW-Source: 2007-11/txt/msg00476.txt.bz2 On Sun, Nov 25, 2007 at 02:08:23PM -0500, Daniel Jacobowitz wrote: >On Sun, Nov 25, 2007 at 12:32:08PM -0500, Christopher Faylor wrote: >> On Sat, Nov 24, 2007 at 05:47:27PM -0500, Daniel Jacobowitz wrote: >> >On Sat, Nov 24, 2007 at 04:07:08PM -0500, Christopher Faylor wrote: >> >> > if (!target_read_string >> >> >- ((CORE_ADDR) current_event.u.DebugString.lpDebugStringData, &s, 1024, >> >> >0) >> >> >+ ((CORE_ADDR) addr, &s, 1024, 0) >> >> >> >> How can coercing something to uintptr_t and then to CORE_ADDR achieve >> >> anything? How does the double coercion help? >> > >> >Just the warning. CORE_ADDR will be long long, >> >current_event.u.DebugString.lpDebugStringData will apparently be a pointer. >> >> And the warning is? > >Cast from pointer to integer of different size. Casts are the way >we've handled it elsewhere in GDB, but I wouldn't complain about a >wrapper; casting host pointers to CORE_ADDRs is an action we try to >keep to a minimum anyway. I wouldn't mind a double cast either, if there is precedent for that. cgf