On Sun, Nov 4, 2012 at 4:02 PM, Uros Bizjak <ubiz...@gmail.com> wrote: > On Sun, Nov 4, 2012 at 3:58 PM, Oleg Endo <oleg.e...@t-online.de> 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. Uros.
