This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH, i386, MPX 1/X] Support of Intel MPX ISA. 2/2 New registers and instructions
- From: Uros Bizjak <ubizjak at gmail dot com>
- To: Ilya Enkovich <enkovich dot gnu at gmail dot com>
- Cc: "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>, "Joseph S. Myers" <joseph at codesourcery dot com>, Jakub Jelinek <jakub at redhat dot com>, Areg Melik-Adamyan <areg dot melikadamyan at gmail dot com>
- Date: Thu, 24 Oct 2013 13:17:53 +0200
- Subject: Re: [PATCH, i386, MPX 1/X] Support of Intel MPX ISA. 2/2 New registers and instructions
- Authentication-results: sourceware.org; auth=none
- References: <CAMbmDYb2K+bm0n7N8xnuGDOVezBMVqD3NuAOu0Qyg0UwpQ+tWA at mail dot gmail dot com> <CAMbmDYa+PMNQJCvhPDsPb-o5iaxcJmCJE2JFyaqJYS+-HM56Vw at mail dot gmail dot com> <CAMbmDYb9eS4K4gwpA04vEoDWJ50HwwgP76FE5b1vn=SqNgdTnQ at mail dot gmail dot com> <CAFULd4Z1Fzur6oSMwXpJKoAtv1BGUX8LFPNjzFMM2Mi1sZUpPQ at mail dot gmail dot com> <CAMbmDYYn+B7PVA+bmvuqmAxF9ApR6kSxe84t0zTmW+55xHbsUw at mail dot gmail dot com> <CAFULd4ZHbzk6U3jdmH-SHT0vyu-DZZa82mCKTk2BdvhSo2VLCQ at mail dot gmail dot com> <20130917084103 dot GA12707 at msticlxl57 dot ims dot intel dot com> <CAFULd4asw5eBU6y7rN9-54Bw44mDZrwXbZ-tZN49H4UtpKWp8g at mail dot gmail dot com> <20131001074145 dot GA40479 at msticlxl57 dot ims dot intel dot com> <CAFULd4a1xx9dw4Gcc9+vnqO1DY-Y6xrKce50u0gjL9UL73wbxQ at mail dot gmail dot com> <20131024100605 dot GB30896 at msticlxl57 dot ims dot intel dot com>
On Thu, Oct 24, 2013 at 12:06 PM, Ilya Enkovich <enkovich.gnu@gmail.com> wrote:
> On 01 Oct 20:00, Uros Bizjak wrote:
>>
>> This is OK for mainline, on the condition that target independent part
>> is approved and committed first.
>>
>> Thanks,
>> Uros.
>
> Thanks for review!
>
> Attached is a version to be committed. The only difference from the previous one is BOUND_MODE renamed to POINTER_BOUNDS_MODE due to changes in target independent part. ChangeLog was not modified.
I think you missed a couple of length -> length_nobnd updates:
@@ -11635,7 +11680,12 @@
[(simple_return)
(unspec [(const_int 0)] UNSPEC_REP)]
"reload_completed"
- "rep%; ret"
+{
+ if (ix86_bnd_prefixed_insn_p (insn))
+ return "%!ret";
+
+ return "rep%; ret";
+}
[(set_attr "length" "2")
(set_attr "atom_unit" "jeu")
(set_attr "length_immediate" "0")
and possibly here:
@@ -11186,7 +11231,7 @@
(define_insn "*indirect_jump"
[(set (pc) (match_operand:W 0 "indirect_branch_operand" "rw"))]
""
- "jmp\t%A0"
+ "%!jmp\t%A0"
[(set_attr "type" "ibr")
(set_attr "length_immediate" "0")])
@@ -11235,7 +11280,7 @@
[(set (pc) (match_operand:W 0 "indirect_branch_operand" "rw"))
(use (label_ref (match_operand 1)))]
""
- "jmp\t%A0"
+ "%!jmp\t%A0"
[(set_attr "type" "ibr")
(set_attr "length_immediate" "0")])
Uros.