On Sun, 2014-11-30 at 20:08 +0100, Oleg Endo wrote:
> Hi,
> 
> In libgcc/lib1funcs.S the usage of the __SHMEDIA__ macro seems to
> mismatch the setup in sh/sh-c.c (sh_cpu_cpp_builtins).  __SHMEDIA__ is
> always defined when building for SH5 / SH64, even when the subtarget is
> SHcompact.  Thus, when building for SHcompact testing '#ifdef
> __SHMEDIA__' will result in the wrong answer and try to compile SHmedia
> code which doesn't work.  Testing the value of __SHMEDIA__ seems to fix
> the issue.
> Tested with 'make all' for sh-elf and sh64-elf, which builds various
> multilib versions of libgcc.  Committed as r218190.
> 
> Cheers,
> Oleg
> 
> libgcc/ChangeLog:
>       * config/sh/lib1funcs.S: Check value of __SHMEDIA__ instead of checking
>       whether it's defined.

Turns out there was already a PR 55351 for that.
Committed to 4.9 (r218544) and 4.8 (r218545).

Cheers,
Oleg

Reply via email to