On Wed, Apr 27, 2016 at 05:54:42AM -0700, H.J. Lu wrote:
> On Thu, Apr 21, 2016 at 11:29 AM, Uros Bizjak <ubiz...@gmail.com> wrote:
> > On Thu, Apr 21, 2016 at 7:46 PM, H.J. Lu <hongjiu...@intel.com> wrote:
> >> r231923 has
> >>
> >>  ;; Test for a valid operand for a call instruction.
> >>  ;; Allow constant call address operands in Pmode only.
> >>  (define_special_predicate "call_insn_operand"
> >>    (ior (match_test "constant_call_address_operand
> >>                      (op, mode == VOIDmode ? mode : Pmode)")
> >>         (match_operand 0 "call_register_no_elim_operand")
> >> -       (and (not (match_test "TARGET_X32"))
> >> -           (match_operand 0 "memory_operand"))))
> >> +       (ior (and (not (match_test "TARGET_X32"))
> >> +                (match_operand 0 "sibcall_memory_operand"))
> >>                                    ^^^^^^^^^^^^^^^^^^^^^^^ A typo.
> >> +           (and (match_test "TARGET_X32 && Pmode == DImode")
> >> +                (match_operand 0 "GOT_memory_operand")))))
> >>
> >> "sibcall_memory_operand" should be "memory_operand".
> >>
> >> OK for trunk and 6 branch if there is no regression on x86-64?
> >
> > OK everywhere, but needs RM's approval for branch.
> 
> OK to backport for GCC 6 branch?

Okay.

        Jakub

Reply via email to