* [PATCH] x86: Remove "%!" before ret
@ 2021-11-17 19:44 H.J. Lu
2021-11-17 19:46 ` Uros Bizjak
0 siblings, 1 reply; 4+ messages in thread
From: H.J. Lu @ 2021-11-17 19:44 UTC (permalink / raw)
To: gcc-patches
Before MPX was removed, "%!" was mapped to
case '!':
if (ix86_bnd_prefixed_insn_p (current_output_insn))
fputs ("bnd ", file);
return;
After CET was added and MPX was removed, "%!" was mapped to
case '!':
if (ix86_notrack_prefixed_insn_p (current_output_insn))
fputs ("notrack ", file);
return;
ix86_notrack_prefixed_insn_p always returns false on ret since the
notrack prefix is only for indirect branches. Remove the unused "%!"
before ret.
PR target/103307
* config/i386/i386.c (ix86_code_end): Remove "%!" before ret.
(ix86_output_function_return): Likewise.
* config/i386/i386.md (simple_return_pop_internal): Likewise.
---
gcc/config/i386/i386.c | 4 ++--
gcc/config/i386/i386.md | 2 +-
2 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/gcc/config/i386/i386.c b/gcc/config/i386/i386.c
index 73c4d5115bb..95d238e9efc 100644
--- a/gcc/config/i386/i386.c
+++ b/gcc/config/i386/i386.c
@@ -6116,7 +6116,7 @@ ix86_code_end (void)
xops[0] = gen_rtx_REG (Pmode, regno);
xops[1] = gen_rtx_MEM (Pmode, stack_pointer_rtx);
output_asm_insn ("mov%z0\t{%1, %0|%0, %1}", xops);
- output_asm_insn ("%!ret", NULL);
+ output_asm_insn ("ret", NULL);
final_end_function ();
init_insn_lengths ();
free_after_compilation (cfun);
@@ -16278,7 +16278,7 @@ ix86_output_function_return (bool long_p)
}
if (!long_p)
- return "%!ret";
+ return "ret";
return "rep%; ret";
}
diff --git a/gcc/config/i386/i386.md b/gcc/config/i386/i386.md
index 73d15de88b2..7b2de60706d 100644
--- a/gcc/config/i386/i386.md
+++ b/gcc/config/i386/i386.md
@@ -14705,7 +14705,7 @@ (define_insn_and_split "simple_return_pop_internal"
[(simple_return)
(use (match_operand:SI 0 "const_int_operand"))]
"reload_completed"
- "%!ret\t%0"
+ "ret\t%0"
"&& cfun->machine->function_return_type != indirect_branch_keep"
[(const_int 0)]
"ix86_split_simple_return_pop_internal (operands[0]); DONE;"
--
2.33.1
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] x86: Remove "%!" before ret
2021-11-17 19:44 [PATCH] x86: Remove "%!" before ret H.J. Lu
@ 2021-11-17 19:46 ` Uros Bizjak
2021-11-17 20:32 ` [PATCH v2] " H.J. Lu
0 siblings, 1 reply; 4+ messages in thread
From: Uros Bizjak @ 2021-11-17 19:46 UTC (permalink / raw)
To: H.J. Lu; +Cc: gcc-patches
On Wed, Nov 17, 2021 at 8:44 PM H.J. Lu <hjl.tools@gmail.com> wrote:
>
> Before MPX was removed, "%!" was mapped to
>
> case '!':
> if (ix86_bnd_prefixed_insn_p (current_output_insn))
> fputs ("bnd ", file);
> return;
>
> After CET was added and MPX was removed, "%!" was mapped to
>
> case '!':
> if (ix86_notrack_prefixed_insn_p (current_output_insn))
> fputs ("notrack ", file);
> return;
>
> ix86_notrack_prefixed_insn_p always returns false on ret since the
> notrack prefix is only for indirect branches. Remove the unused "%!"
> before ret.
>
> PR target/103307
> * config/i386/i386.c (ix86_code_end): Remove "%!" before ret.
> (ix86_output_function_return): Likewise.
> * config/i386/i386.md (simple_return_pop_internal): Likewise.
> ---
> gcc/config/i386/i386.c | 4 ++--
> gcc/config/i386/i386.md | 2 +-
> 2 files changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/gcc/config/i386/i386.c b/gcc/config/i386/i386.c
> index 73c4d5115bb..95d238e9efc 100644
> --- a/gcc/config/i386/i386.c
> +++ b/gcc/config/i386/i386.c
> @@ -6116,7 +6116,7 @@ ix86_code_end (void)
> xops[0] = gen_rtx_REG (Pmode, regno);
> xops[1] = gen_rtx_MEM (Pmode, stack_pointer_rtx);
> output_asm_insn ("mov%z0\t{%1, %0|%0, %1}", xops);
> - output_asm_insn ("%!ret", NULL);
> + output_asm_insn ("ret", NULL);
This can use fputs.
Uros.
> final_end_function ();
> init_insn_lengths ();
> free_after_compilation (cfun);
> @@ -16278,7 +16278,7 @@ ix86_output_function_return (bool long_p)
> }
>
> if (!long_p)
> - return "%!ret";
> + return "ret";
>
> return "rep%; ret";
> }
> diff --git a/gcc/config/i386/i386.md b/gcc/config/i386/i386.md
> index 73d15de88b2..7b2de60706d 100644
> --- a/gcc/config/i386/i386.md
> +++ b/gcc/config/i386/i386.md
> @@ -14705,7 +14705,7 @@ (define_insn_and_split "simple_return_pop_internal"
> [(simple_return)
> (use (match_operand:SI 0 "const_int_operand"))]
> "reload_completed"
> - "%!ret\t%0"
> + "ret\t%0"
> "&& cfun->machine->function_return_type != indirect_branch_keep"
> [(const_int 0)]
> "ix86_split_simple_return_pop_internal (operands[0]); DONE;"
> --
> 2.33.1
>
^ permalink raw reply [flat|nested] 4+ messages in thread
* [PATCH v2] x86: Remove "%!" before ret
2021-11-17 19:46 ` Uros Bizjak
@ 2021-11-17 20:32 ` H.J. Lu
2021-11-17 20:34 ` Uros Bizjak
0 siblings, 1 reply; 4+ messages in thread
From: H.J. Lu @ 2021-11-17 20:32 UTC (permalink / raw)
To: Uros Bizjak; +Cc: gcc-patches
[-- Attachment #1: Type: text/plain, Size: 2555 bytes --]
On Wed, Nov 17, 2021 at 11:46 AM Uros Bizjak <ubizjak@gmail.com> wrote:
>
> On Wed, Nov 17, 2021 at 8:44 PM H.J. Lu <hjl.tools@gmail.com> wrote:
> >
> > Before MPX was removed, "%!" was mapped to
> >
> > case '!':
> > if (ix86_bnd_prefixed_insn_p (current_output_insn))
> > fputs ("bnd ", file);
> > return;
> >
> > After CET was added and MPX was removed, "%!" was mapped to
> >
> > case '!':
> > if (ix86_notrack_prefixed_insn_p (current_output_insn))
> > fputs ("notrack ", file);
> > return;
> >
> > ix86_notrack_prefixed_insn_p always returns false on ret since the
> > notrack prefix is only for indirect branches. Remove the unused "%!"
> > before ret.
> >
> > PR target/103307
> > * config/i386/i386.c (ix86_code_end): Remove "%!" before ret.
> > (ix86_output_function_return): Likewise.
> > * config/i386/i386.md (simple_return_pop_internal): Likewise.
> > ---
> > gcc/config/i386/i386.c | 4 ++--
> > gcc/config/i386/i386.md | 2 +-
> > 2 files changed, 3 insertions(+), 3 deletions(-)
> >
> > diff --git a/gcc/config/i386/i386.c b/gcc/config/i386/i386.c
> > index 73c4d5115bb..95d238e9efc 100644
> > --- a/gcc/config/i386/i386.c
> > +++ b/gcc/config/i386/i386.c
> > @@ -6116,7 +6116,7 @@ ix86_code_end (void)
> > xops[0] = gen_rtx_REG (Pmode, regno);
> > xops[1] = gen_rtx_MEM (Pmode, stack_pointer_rtx);
> > output_asm_insn ("mov%z0\t{%1, %0|%0, %1}", xops);
> > - output_asm_insn ("%!ret", NULL);
> > + output_asm_insn ("ret", NULL);
>
> This can use fputs.
Fixed. Here is the v2 patch.
> Uros.
>
> > final_end_function ();
> > init_insn_lengths ();
> > free_after_compilation (cfun);
> > @@ -16278,7 +16278,7 @@ ix86_output_function_return (bool long_p)
> > }
> >
> > if (!long_p)
> > - return "%!ret";
> > + return "ret";
> >
> > return "rep%; ret";
> > }
> > diff --git a/gcc/config/i386/i386.md b/gcc/config/i386/i386.md
> > index 73d15de88b2..7b2de60706d 100644
> > --- a/gcc/config/i386/i386.md
> > +++ b/gcc/config/i386/i386.md
> > @@ -14705,7 +14705,7 @@ (define_insn_and_split "simple_return_pop_internal"
> > [(simple_return)
> > (use (match_operand:SI 0 "const_int_operand"))]
> > "reload_completed"
> > - "%!ret\t%0"
> > + "ret\t%0"
> > "&& cfun->machine->function_return_type != indirect_branch_keep"
> > [(const_int 0)]
> > "ix86_split_simple_return_pop_internal (operands[0]); DONE;"
> > --
> > 2.33.1
> >
--
H.J.
[-- Attachment #2: v2-0001-x86-Remove-before-ret.patch --]
[-- Type: text/x-patch, Size: 2249 bytes --]
From 594391d282f0066cb046dd06062e3efad8c74a08 Mon Sep 17 00:00:00 2001
From: "H.J. Lu" <hjl.tools@gmail.com>
Date: Wed, 17 Nov 2021 11:41:12 -0800
Subject: [PATCH v2] x86: Remove "%!" before ret
Before MPX was removed, "%!" was mapped to
case '!':
if (ix86_bnd_prefixed_insn_p (current_output_insn))
fputs ("bnd ", file);
return;
After CET was added and MPX was removed, "%!" was mapped to
case '!':
if (ix86_notrack_prefixed_insn_p (current_output_insn))
fputs ("notrack ", file);
return;
ix86_notrack_prefixed_insn_p always returns false on ret since the
notrack prefix is only for indirect branches. Remove the unused "%!"
before ret.
PR target/103307
* config/i386/i386.c (ix86_code_end): Remove "%!" before ret.
(ix86_output_function_return): Likewise.
* config/i386/i386.md (simple_return_pop_internal): Likewise.
---
gcc/config/i386/i386.c | 4 ++--
gcc/config/i386/i386.md | 2 +-
2 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/gcc/config/i386/i386.c b/gcc/config/i386/i386.c
index c9129ae25e4..a5bfb9efca9 100644
--- a/gcc/config/i386/i386.c
+++ b/gcc/config/i386/i386.c
@@ -6115,7 +6115,7 @@ ix86_code_end (void)
xops[0] = gen_rtx_REG (Pmode, regno);
xops[1] = gen_rtx_MEM (Pmode, stack_pointer_rtx);
output_asm_insn ("mov%z0\t{%1, %0|%0, %1}", xops);
- output_asm_insn ("%!ret", NULL);
+ fputs ("\tret\n", asm_out_file);
final_end_function ();
init_insn_lengths ();
free_after_compilation (cfun);
@@ -16273,7 +16273,7 @@ ix86_output_function_return (bool long_p)
}
if (!long_p)
- return "%!ret";
+ return "ret";
return "rep%; ret";
}
diff --git a/gcc/config/i386/i386.md b/gcc/config/i386/i386.md
index 73d15de88b2..7b2de60706d 100644
--- a/gcc/config/i386/i386.md
+++ b/gcc/config/i386/i386.md
@@ -14705,7 +14705,7 @@ (define_insn_and_split "simple_return_pop_internal"
[(simple_return)
(use (match_operand:SI 0 "const_int_operand"))]
"reload_completed"
- "%!ret\t%0"
+ "ret\t%0"
"&& cfun->machine->function_return_type != indirect_branch_keep"
[(const_int 0)]
"ix86_split_simple_return_pop_internal (operands[0]); DONE;"
--
2.33.1
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH v2] x86: Remove "%!" before ret
2021-11-17 20:32 ` [PATCH v2] " H.J. Lu
@ 2021-11-17 20:34 ` Uros Bizjak
0 siblings, 0 replies; 4+ messages in thread
From: Uros Bizjak @ 2021-11-17 20:34 UTC (permalink / raw)
To: H.J. Lu; +Cc: gcc-patches
On Wed, Nov 17, 2021 at 9:33 PM H.J. Lu <hjl.tools@gmail.com> wrote:
>
> On Wed, Nov 17, 2021 at 11:46 AM Uros Bizjak <ubizjak@gmail.com> wrote:
> >
> > On Wed, Nov 17, 2021 at 8:44 PM H.J. Lu <hjl.tools@gmail.com> wrote:
> > >
> > > Before MPX was removed, "%!" was mapped to
> > >
> > > case '!':
> > > if (ix86_bnd_prefixed_insn_p (current_output_insn))
> > > fputs ("bnd ", file);
> > > return;
> > >
> > > After CET was added and MPX was removed, "%!" was mapped to
> > >
> > > case '!':
> > > if (ix86_notrack_prefixed_insn_p (current_output_insn))
> > > fputs ("notrack ", file);
> > > return;
> > >
> > > ix86_notrack_prefixed_insn_p always returns false on ret since the
> > > notrack prefix is only for indirect branches. Remove the unused "%!"
> > > before ret.
> > >
> > > PR target/103307
> > > * config/i386/i386.c (ix86_code_end): Remove "%!" before ret.
> > > (ix86_output_function_return): Likewise.
> > > * config/i386/i386.md (simple_return_pop_internal): Likewise.
> > > ---
> > > gcc/config/i386/i386.c | 4 ++--
> > > gcc/config/i386/i386.md | 2 +-
> > > 2 files changed, 3 insertions(+), 3 deletions(-)
> > >
> > > diff --git a/gcc/config/i386/i386.c b/gcc/config/i386/i386.c
> > > index 73c4d5115bb..95d238e9efc 100644
> > > --- a/gcc/config/i386/i386.c
> > > +++ b/gcc/config/i386/i386.c
> > > @@ -6116,7 +6116,7 @@ ix86_code_end (void)
> > > xops[0] = gen_rtx_REG (Pmode, regno);
> > > xops[1] = gen_rtx_MEM (Pmode, stack_pointer_rtx);
> > > output_asm_insn ("mov%z0\t{%1, %0|%0, %1}", xops);
> > > - output_asm_insn ("%!ret", NULL);
> > > + output_asm_insn ("ret", NULL);
> >
> > This can use fputs.
>
> Fixed. Here is the v2 patch.
OK.
Thanks,
Uros.
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2021-11-17 20:34 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-11-17 19:44 [PATCH] x86: Remove "%!" before ret H.J. Lu
2021-11-17 19:46 ` Uros Bizjak
2021-11-17 20:32 ` [PATCH v2] " H.J. Lu
2021-11-17 20:34 ` Uros Bizjak
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).