On Jul 7, 2011, at 1:00 PM, Jeff Law wrote:

> -----BEGIN PGP SIGNED MESSAGE-----
> Hash: SHA1
> 
> On 07/07/11 10:58, Paul Koning wrote:
>> 
>> On Jul 7, 2011, at 11:38 AM, Bernd Schmidt wrote:
>> 
>>> ...
>>> 
>>>> It'd also be nice to get rid of all these big blocks of code that are
>>>> conditional on preprocessor macros, but I realise you're just following
>>>> existing practice in the surrounding code, so again it can be left to
>>>> a future cleanup.
>>> 
>>> Yeah, this function is quite horrid - so many different paths through it.
>>> 
>>> However, it looks like the only target without HAVE_prologue is actually
>>> pdp11, so we're carrying some unnecessary baggage for purely
>>> retrocomputing purposes. Paul, can you fix that?
>> 
>> Sure, but...  I searched for HAVE_prologue and I can't find any place that 
>> set it.  There are tests for it, but I see nothing that defines it (other 
>> than df-scan.c which defines it as zero if it's not defined, not sure what 
>> the point of that is).
>> 
>> I must be missing something...
> Isn't it defined by the insn-foo generators based on the existence of a
> prologue/epilogue insn in the MD file?

Thanks, that must be what I was missing.  So someone is generating HAVE_%s, and 
that's why grep didn't find HAVE_prologue?

>From a note by Richard Henderson (June 30, 2011) it sounds like rs6000 is the 
>other platform that still generates asm prologues.  But yes, I said I would do 
>this.  It sounds like doing it soon would help Bernd a lot.  Let me try to 
>accelerate it.

        paul


Reply via email to