This is the mail archive of the gcc@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: pa indirect_jump instruction


On Sun, Jul 05, 2015 at 09:11:23AM +0100, Richard Sandiford wrote:
> Trevor Saunders <tbsaunde@tbsaunde.org> writes:
> > On Tue, Jun 30, 2015 at 09:53:31PM +0100, Richard Sandiford wrote:
> >> I have a series of patches to convert all non-optab instructions to
> >> the target-insns.def interface.  config-list.mk showed up one problem
> >> though.  The pa indirect_jump pattern is:
> >> 
> >> ;;; Hope this is only within a function...
> >> (define_insn "indirect_jump"
> >>   [(set (pc) (match_operand 0 "register_operand" "r"))]
> >>   "GET_MODE (operands[0]) == word_mode"
> >>   "bv%* %%r0(%0)"
> >>   [(set_attr "type" "branch")
> >>    (set_attr "length" "4")])
> >> 
> >> so the C condition depends on operands[], which isn't usually allowed
> >> for named patterns.  We get away with it at the moment because we only
> >> test for the existence of HAVE_indirect_jump, not its value:
> >
> > yeah, I hit this a while ago and filed bug 66114.  It looks like I had
> > trouble with fr30 too, is that fixed now?
> 
> Hmm, seems not.  The fr30 build stopped earlier for me due to a warning
> turned error.  I suppose I should really have fixed all the warnings shown
> by config-list.mk before doing this stuff...

yeah, that's certainly a problem worth working on, but there's certainly
something to be said for not going too far down the yak shaving rabbit
whole.

Trev

far dow
> 
> Thanks,
> Richard


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