public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
* [PATCH] [PR42093] [arm] [thumb2] disable tree-dce for test
@ 2022-12-02  9:26 Alexandre Oliva
  2023-01-12 14:43 ` Richard Earnshaw
  0 siblings, 1 reply; 2+ messages in thread
From: Alexandre Oliva @ 2022-12-02  9:26 UTC (permalink / raw)
  To: gcc-patches; +Cc: Rainer Orth, Mike Stump


CD-DCE introduces blocks to share common PHI nodes, which replaces a
backwards branch that used to prevent the thumb2 jump table shortening
that PR42093 tested for.  In order to keep on testing that the
backward branch prevents the jumptable shortening, disable tree-dce.

Regstraped on x86_64-linux-gnu, also tested with crosses to riscv64-elf
and arm-eabi.  Ok to install?


for  gcc/testsuite/ChangeLog

	PR target/42093
	* gcc.target/arm/pr42093.c: Disable tree-dce.
---
 gcc/testsuite/gcc.target/arm/pr42093.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/gcc/testsuite/gcc.target/arm/pr42093.c b/gcc/testsuite/gcc.target/arm/pr42093.c
index 7ba2f933eef81..69b1470607c7f 100644
--- a/gcc/testsuite/gcc.target/arm/pr42093.c
+++ b/gcc/testsuite/gcc.target/arm/pr42093.c
@@ -1,4 +1,4 @@
-/* { dg-options "-mthumb -O2 -fno-reorder-blocks" }  */
+/* { dg-options "-mthumb -O2 -fno-reorder-blocks -fno-tree-dce" }  */
 /* { dg-require-effective-target arm_thumb2_ok } */
 /* { dg-final { scan-assembler-not "tbb" } } */
 /* { dg-final { scan-assembler-not "tbh" } } */

-- 
Alexandre Oliva, happy hacker                https://FSFLA.org/blogs/lxo/
   Free Software Activist                       GNU Toolchain Engineer
Disinformation flourishes because many people care deeply about injustice
but very few check the facts.  Ask me about <https://stallmansupport.org>

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [PATCH] [PR42093] [arm] [thumb2] disable tree-dce for test
  2022-12-02  9:26 [PATCH] [PR42093] [arm] [thumb2] disable tree-dce for test Alexandre Oliva
@ 2023-01-12 14:43 ` Richard Earnshaw
  0 siblings, 0 replies; 2+ messages in thread
From: Richard Earnshaw @ 2023-01-12 14:43 UTC (permalink / raw)
  To: Alexandre Oliva, gcc-patches; +Cc: Rainer Orth, Mike Stump



On 02/12/2022 09:26, Alexandre Oliva via Gcc-patches wrote:
> 
> CD-DCE introduces blocks to share common PHI nodes, which replaces a
> backwards branch that used to prevent the thumb2 jump table shortening
> that PR42093 tested for.  In order to keep on testing that the
> backward branch prevents the jumptable shortening, disable tree-dce.
> 
> Regstraped on x86_64-linux-gnu, also tested with crosses to riscv64-elf
> and arm-eabi.  Ok to install?
> 
>  > for  gcc/testsuite/ChangeLog
> 
> 	PR target/42093
> 	* gcc.target/arm/pr42093.c: Disable tree-dce.

OK.

R.

> ---
>   gcc/testsuite/gcc.target/arm/pr42093.c |    2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/gcc/testsuite/gcc.target/arm/pr42093.c b/gcc/testsuite/gcc.target/arm/pr42093.c
> index 7ba2f933eef81..69b1470607c7f 100644
> --- a/gcc/testsuite/gcc.target/arm/pr42093.c
> +++ b/gcc/testsuite/gcc.target/arm/pr42093.c
> @@ -1,4 +1,4 @@
> -/* { dg-options "-mthumb -O2 -fno-reorder-blocks" }  */
> +/* { dg-options "-mthumb -O2 -fno-reorder-blocks -fno-tree-dce" }  */
>   /* { dg-require-effective-target arm_thumb2_ok } */
>   /* { dg-final { scan-assembler-not "tbb" } } */
>   /* { dg-final { scan-assembler-not "tbh" } } */
> 

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2023-01-12 14:43 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-12-02  9:26 [PATCH] [PR42093] [arm] [thumb2] disable tree-dce for test Alexandre Oliva
2023-01-12 14:43 ` Richard Earnshaw

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).