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