This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: PR debug/41248 [VTA] deal with Pmode != ptr_mode in expand_debug_expr


On Sat, Sep 5, 2009 at 12:28 AM, Steve Ellcey<sje@cup.hp.com> wrote:
> On Fri, 2009-09-04 at 17:33 -0300, Alexandre Oliva wrote:
>> On Sep ?4, 2009, Steve Ellcey <sje@cup.hp.com> wrote:
>>
>> > On Thu, 2009-09-03 at 22:16 -0300, Alexandre Oliva wrote:
>> >> Hi, Steve,
>> >>
>> >> Is this ok to install if it passes bootstrap testing? ?Could you please
>> >> give it a try on ia64-hpux? ?Thanks in advance,
>>
>> > I changed Pmode to ptr_mode in the assert at line 2450 in
>> > expand_debug_expr and everything compiled but when I went to run genmode
>> > that was built with the new compiler (during bootstrap) the genmode
>> > aborted in htab_create_alloc, called from main.
>>
>> Hmm... ?I find it unlikely that this second failure is in any way
>> related with VTA, but you could confirm that it isn't setting
>> BOOT_CFLAGS='-g -O2 -fno-var-tracking-assignments', or say
>> BOOT_CFLAGS='-g -O2 -fcompare-debug=-gtoggle', then wiping out the
>> stage2 build and starting over.
>>
>> Here's the updated patch, that accepts ptr_mode in two other locations
>> where we used to require Pmode.
>
> You are right, the genmode failure does not appear to be related to your
> change, it started with r151350 (Michael Matz's expand from tuple
> patch). ?Michael, have you had any reports of problems with this change?

I'm not aware of any reported problems with that change, but it's certainly
a good guess if the arch has specialities different from x86 and ppc.

Richard.

> Steve Ellcey
> sje@cup.hp.com
>
>


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]