This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: Statically propagate basic blocks which are likely executed 0 times
> Hi Honza & Christophe,
>
> I have tested your suggested fix. It does fix the regression.
> Here is a simple patch for it.
>
> After r249013, die () and dump_stack () are both in cold section. This makes
> the compiler generate bl instruction for the function call, instead of
> honoring the -mlong-calls option.
>
> This patch changes the dump_stack function call conditional, which fixes the
> regression.
>
> Okay to commit?
>
> Regards,
> Renlin
>
> gcc/testsuite/ChangeLog:
>
> 2017-06-12 Renlin Li <renlin.li@arm.com>
>
> * gcc.target/arm/cold-lc.c: Update coding style, call dump_stack
> conditionally.
Looks OK to me. I think it does not change purpose of the testcase ;)
Honza