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