This is a trivial patch. However, without it, the addressing mode cost is incorrectly calculated, since the cost intended for HI end up being used for SI on A57.
I'd appreciate your considering this patch fixing this issue. Thank you, -- Evandro Menezes Austin, USA e.mene...@samsung.com +1-512-425-3365 -----Original Message----- From: Evandro Menezes [mailto:e.mene...@samsung.com] Sent: Tuesday, August 19, 2014 14:40 To: 'James Greenhalgh' Cc: 'gcc-patches@gcc.gnu.org'; 'James Greenhalgh'; 'richard.earns...@arm.com'; 'marcus.shawcr...@arm.com' Subject: RE: [PATCH, AArch64] Fix typo Ping, pretty please. -- Evandro Menezes Austin, USA e.mene...@samsung.com +1-512-425-3365 -----Original Message----- From: Evandro Menezes [mailto:e.mene...@samsung.com] Sent: Monday, August 18, 2014 10:02 To: 'James Greenhalgh' Cc: 'gcc-patches@gcc.gnu.org'; 'James Greenhalgh'; 'richard.earns...@arm.com'; 'marcus.shawcr...@arm.com' Subject: RE: [PATCH, AArch64] Fix typo Ping. -- Evandro Menezes Austin, USA e.mene...@samsung.com +1-512-425-3365 -----Original Message----- From: Evandro Menezes [mailto:e.mene...@samsung.com] Sent: Friday, August 15, 2014 14:55 To: 'James Greenhalgh' Cc: 'gcc-patches@gcc.gnu.org'; 'James Greenhalgh'; 'richard.earns...@arm.com'; 'marcus.shawcr...@arm.com' Subject: RE: [PATCH, AArch64] Fix typo Thanks for the review. -- Evandro Menezes Austin, USA e.mene...@samsung.com +1-512-425-3365 -----Original Message----- From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches-ow...@gcc.gnu.org] On Behalf Of James Greenhalgh Sent: Friday, August 15, 2014 11:36 To: Evandro Menezes Cc: gcc-patches@gcc.gnu.org; 'James Greenhalgh'; richard.earns...@arm.com; marcus.shawcr...@arm.com Subject: Re: [PATCH, AArch64] Fix typo On Fri, Aug 15, 2014 at 05:24:58PM +0100, Evandro Menezes wrote: > I tripped at a typo that goes undetected because the macro NAMED_PARAM > doesn't apply in the absence of designated initializers. > > Since struct scale_addr_mode_cost has the cost for DI, but not for QI, > the instances of struct cpu_addrcost_table are not initialized as > intended due to the different order of the structure members. Thanks for spotting and fixing this. The ChangeLog entry should be added to gcc/ChangeLog, and should look like this: 2014-08-14 Evandro Menezes <e.mene...@samsung.com> * config/aarch64/aarch64.c (generic_addrcost_table): Initialize elements in the correct order. (cortexa57_addrcost_table): Likewise. My fixes were: * Two spaces between your name and email address. * Name the structure/function/thing changed. * Set the path relative to the ChangeLog being modified. Otherwise, this patch looks correct to me. However, you will need approval from an AArch64 port maintainer (For AArch64 this is Richard Earnshaw or Marcus Shawcroft - both added to CC). Thanks, James
aarch64.diff
Description: Binary data
