public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
* [AArch64-4.7] Fix warning: TARGET_FIXED_CONDITION_CODE_REGS redefined.
@ 2013-03-01 10:49 James Greenhalgh
  2013-03-01 12:44 ` Marcus Shawcroft
  0 siblings, 1 reply; 2+ messages in thread
From: James Greenhalgh @ 2013-03-01 10:49 UTC (permalink / raw)
  To: gcc-patches; +Cc: marcus.shawcroft

[-- Attachment #1: Type: text/plain, Size: 627 bytes --]


Hi,

config/aarch64/aarch64.c attempts to do this:

> #undef TARGET_FIXED_CONDITION_CODE_REG
> #define TARGET_FIXED_CONDITION_CODE_REGS aarch64_fixed_condition_code_regs

This patch adds the S to the end of TARGET_FIXED_CONDITION_CODE_REG
clearing a warning for redefining TARGET_FIXED_CONDITION_REGS

Regression tested on aarch64-none-elf with no issues and checked
to ensure that the warning is cleared.

OK for aarch64-4.7-branch?

Thanks,
James

---
gcc/

2013-03-01  James Greenhalgh  <james.greenhalgh@arm.com>

	* config/aarch64/aarch64.c:
	Fix typo in `#undef TARGET_FIXED_CONDITION_CODE_REGS'

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: 0001-AArch64-4.7-Fix-warning-TARGET_FIXED_CONDITION_CODE_.patch --]
[-- Type: text/x-patch;  name=0001-AArch64-4.7-Fix-warning-TARGET_FIXED_CONDITION_CODE_.patch, Size: 568 bytes --]

diff --git a/gcc/config/aarch64/aarch64.c b/gcc/config/aarch64/aarch64.c
index b554a0e..5bc6f84 100644
--- a/gcc/config/aarch64/aarch64.c
+++ b/gcc/config/aarch64/aarch64.c
@@ -7819,7 +7819,7 @@ aarch64_vectorize_vec_perm_const_ok (enum machine_mode vmode,
 #undef TARGET_EXPAND_BUILTIN_VA_START
 #define TARGET_EXPAND_BUILTIN_VA_START aarch64_expand_builtin_va_start
 
-#undef TARGET_FIXED_CONDITION_CODE_REG
+#undef TARGET_FIXED_CONDITION_CODE_REGS
 #define TARGET_FIXED_CONDITION_CODE_REGS aarch64_fixed_condition_code_regs
 
 #undef TARGET_FUNCTION_ARG

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

* Re: [AArch64-4.7] Fix warning: TARGET_FIXED_CONDITION_CODE_REGS redefined.
  2013-03-01 10:49 [AArch64-4.7] Fix warning: TARGET_FIXED_CONDITION_CODE_REGS redefined James Greenhalgh
@ 2013-03-01 12:44 ` Marcus Shawcroft
  0 siblings, 0 replies; 2+ messages in thread
From: Marcus Shawcroft @ 2013-03-01 12:44 UTC (permalink / raw)
  To: James Greenhalgh; +Cc: gcc-patches, marcus.shawcroft

OK

On 1 March 2013 10:49, James Greenhalgh <james.greenhalgh@arm.com> wrote:
>
> Hi,
>
> config/aarch64/aarch64.c attempts to do this:
>
>> #undef TARGET_FIXED_CONDITION_CODE_REG
>> #define TARGET_FIXED_CONDITION_CODE_REGS aarch64_fixed_condition_code_regs
>
> This patch adds the S to the end of TARGET_FIXED_CONDITION_CODE_REG
> clearing a warning for redefining TARGET_FIXED_CONDITION_REGS
>
> Regression tested on aarch64-none-elf with no issues and checked
> to ensure that the warning is cleared.
>
> OK for aarch64-4.7-branch?
>
> Thanks,
> James
>
> ---
> gcc/
>
> 2013-03-01  James Greenhalgh  <james.greenhalgh@arm.com>
>
>         * config/aarch64/aarch64.c:
>         Fix typo in `#undef TARGET_FIXED_CONDITION_CODE_REGS'

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

end of thread, other threads:[~2013-03-01 12:44 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-03-01 10:49 [AArch64-4.7] Fix warning: TARGET_FIXED_CONDITION_CODE_REGS redefined James Greenhalgh
2013-03-01 12:44 ` Marcus Shawcroft

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