On 4/18/2016 6:11 AM, Oleg Endo wrote:
On Sun, 2016-04-17 at 13:33 -0500, Joel Sherrill wrote:
Thanks for the quick and thorough reply.
This doesn't happen with GCC 4.9 which we are using on our newest
release branch. With any luck your work will be in gcc 7 before we
make another release branch.
Since I stated that, we decided to use the 6.1 branch for a while.
So I decided to look at config/sh/linux.h and see what it was doing.
Copying if on the 6.1 branch seemed liked an option. But it only
appears to address SH3 and SH1 for atomics. What about an implicit
atomic for SH2 or SH4?
/* Set default atomic model if it hasn't been specified. */ \
if (global_options_set.x_sh_atomic_model_str == 0) \
{ \
if (TARGET_SH3) \
sh_atomic_model_str = "soft-gusa"; \
else if (TARGET_SH1) \
sh_atomic_model_str = "soft-imask"; \
}
That's probably because of this commit:
https://gcc.gnu.org/viewcvs?rev=220094&root=gcc&view=rev
Possibly. This BSP is compiled with -m4 -ml
Is there a ticket for your plan I should add myself to to track this?
No, for that particular issue there's no ticket. I can put you in CC
when I send around/commit the patch, if that helps.
Please do. I may just leave this as a breakage and let you fix it.
AFAIK no one is really complaining that it is is broken on our
development master.
Cheers,
Oleg
\
--joel
