Re: [PATCH arm/embedded-4_7-branch]fix fialure of mtune option

Re: [PATCH arm/embedded-4_7-branch]fix fialure of mtune option

[Ye Joey]

OK to embedded-4_7-branch.

Thanks,
Joey

On Thu, Jun 6, 2013 at 9:43 AM, Bin Cheng <bin.cheng@arm.com> wrote:
> Hi,
> This patch fixes failure of command "arm-none-eabi-gcc -O2 -mthumb
> -mtune=cortex-m4 ..." on arm/embedded-4_7-branch by removing the assertion
> in arm_cortex_v7m_branch_cost.
>
> Is it OK?
>
> Thanks.
>
> 2013-06-06  Bin Cheng  <bin.cheng@arm.com>
>
>         * config/arm/arm.c (arm_cortex_v7m_branch_cost): Remove assertion
>         on TARGET_32BIT and TARGET_THUMB2.

[PATCH arm/embedded-4_7-branch]fix fialure of mtune option

[Bin Cheng]

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

Hi,
This patch fixes failure of command "arm-none-eabi-gcc -O2 -mthumb
-mtune=cortex-m4 ..." on arm/embedded-4_7-branch by removing the assertion
in arm_cortex_v7m_branch_cost.

Is it OK?

Thanks.

2013-06-06  Bin Cheng  <bin.cheng@arm.com>

	* config/arm/arm.c (arm_cortex_v7m_branch_cost): Remove assertion
	on TARGET_32BIT and TARGET_THUMB2.

[-- Attachment #2: v7m_branch_cost.txt --]
[-- Type: text/plain, Size: 427 bytes --]

Index: gcc/config/arm/arm.c
===================================================================
--- gcc/config/arm/arm.c	(revision 199680)
+++ gcc/config/arm/arm.c	(working copy)
@@ -8760,8 +8760,6 @@ arm_cortex_a5_branch_cost (bool speed_p, bool pred
 static int
 arm_cortex_v7m_branch_cost (bool speed_p, bool predictable_p ATTRIBUTE_UNUSED)
 {
-  gcc_assert (TARGET_32BIT && TARGET_THUMB2);
-
   return 1;
 }