Re: Bootstrap failure on Solaris

[Roman Lechtchinsky <rl at cs dot tu-berlin dot de>]

Hi,

> Hmm, I see, the gen_lowpart is able to handle CONST_INTs,
> because the operation is always well defined,  even when you don't know
> the mode.

Ah, I see.

> Concerning the gen_highpart it didn't handled CONST_INTs correctly, as
> it missed the information of size of the value, but siltenly generated
> wrong code.
> 
> I will send patch fixing the comment. Or perhaps we can simply add the
> mode argument.

Yes, I think adding the mode argument to gen_highpart is the right thing
to do. In fact, I think there are some calls in the compiler proper which
might be passing CONST_INTs to gen_highpart, e.g. in emit_store_flag.
VOIDmode as the mode argument would then indicate that the rtx has a valid
mode which should be used. I'll prepare a patch which implements this
today or tomorrow. I'll only be able to test it on Sparc, though.

> Subreg changes i've made has been quite destabilizing and so I am happy to
> see thinks settling down.

Hmm, I still have to make my T3E port work with them...

> I have prepared second wave of such changes and I would really love to see
> such issues discovered and fixed soon.

Uh-oh ;-)

Bye

Roman