On Sun, Nov 4, 2012 at 4:23 PM, Uros Bizjak <ubiz...@gmail.com> wrote:
>>>> Patch was bootstrapped and regression tested on x86_64-pc-linux-gnu, >>>> with vzeroupper patch [1] applied. >>>> >>>> I have added SH4 maintainer for possible comments. >>> >>> BTW, there are at least two mode-switching SH PRs: 41933 and 49220. >>> I've tried those test cases with the vzeroupper patch [1] applied. >>> Unfortunately, it doesn't change the situation of the two PRs. >> >> True, this patch fixes very specific case involving __builtin_apply >> only. Hopefully it doesn't _break_ something on SH. > > FYI, the testcase from PR41993 involving __builtin_return inserts > vzeroupper at correct place, even with "-O0 -mavx -vzeroupper", so the > ICE seems SH specific. The testcase from PR49220 works OK as well for > all compile flags I have tried, although YMM registers are not used, > and consequently no vzerouppers are inserted. Please see PR41993 for follow-up discussion and possible fix. Uros.
