public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
* [committed] rs6000: Remove typedef for struct rs6000_cost_data
@ 2021-09-13  6:42 Kewen.Lin
  2021-09-13  7:22 ` Segher Boessenkool
  0 siblings, 1 reply; 2+ messages in thread
From: Kewen.Lin @ 2021-09-13  6:42 UTC (permalink / raw)
  To: GCC Patches; +Cc: Segher Boessenkool

Hi,

This patch follows Segher's suggestion here[1] to get rid of 
the typedef, it's pre-approved as [1].

Bootstrapped and regtested on powerpc64le-linux-gnu Power9.

Pushed to trunk as r12-3468.

[1] https://gcc.gnu.org/pipermail/gcc-patches/2021-September/579115.html

BR,
Kewen
-----
gcc/ChangeLog:

	* config/rs6000/rs6000.c (struct rs6000_cost_data): Remove typedef.
	(rs6000_init_cost): Adjust.

--
diff --git a/gcc/config/rs6000/rs6000.c b/gcc/config/rs6000/rs6000.c
index b7ea1483da5..39d428db8e6 100644
--- a/gcc/config/rs6000/rs6000.c
+++ b/gcc/config/rs6000/rs6000.c
@@ -5262,7 +5262,7 @@ rs6000_preferred_simd_mode (scalar_mode mode)
   return word_mode;
 }

-typedef struct _rs6000_cost_data
+struct rs6000_cost_data
 {
   struct loop *loop_info;
   unsigned cost[3];
@@ -5271,7 +5271,7 @@ typedef struct _rs6000_cost_data
   bool vect_nonmem;
   /* Indicates this is costing for the scalar version of a loop or block.  */
   bool costing_for_scalar;
-} rs6000_cost_data;
+};

 /* Test for likely overcommitment of vector hardware resources.  If a
    loop iteration is relatively large, and too large a percentage of
@@ -5337,7 +5337,7 @@ rs6000_density_test (rs6000_cost_data *data)
 static void *
 rs6000_init_cost (struct loop *loop_info, bool costing_for_scalar)
 {
-  rs6000_cost_data *data = XNEW (struct _rs6000_cost_data);
+  rs6000_cost_data *data = XNEW (rs6000_cost_data);
   data->loop_info = loop_info;
   data->cost[vect_prologue] = 0;
   data->cost[vect_body]     = 0;

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

* Re: [committed] rs6000: Remove typedef for struct rs6000_cost_data
  2021-09-13  6:42 [committed] rs6000: Remove typedef for struct rs6000_cost_data Kewen.Lin
@ 2021-09-13  7:22 ` Segher Boessenkool
  0 siblings, 0 replies; 2+ messages in thread
From: Segher Boessenkool @ 2021-09-13  7:22 UTC (permalink / raw)
  To: Kewen.Lin; +Cc: GCC Patches

On Mon, Sep 13, 2021 at 02:42:10PM +0800, Kewen.Lin wrote:
> This patch follows Segher's suggestion here[1] to get rid of 
> the typedef, it's pre-approved as [1].

Thanks!


Segher

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

end of thread, other threads:[~2021-09-13  7:23 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-09-13  6:42 [committed] rs6000: Remove typedef for struct rs6000_cost_data Kewen.Lin
2021-09-13  7:22 ` Segher Boessenkool

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