On Wed, Jan 14, 2015 at 1:14 PM, Ilya Tocar <tocarip.in...@gmail.com> wrote: > On 14 Jan 12:36, Uros Bizjak wrote: >> On Wed, Jan 14, 2015 at 12:18 PM, Ilya Tocar <tocarip.in...@gmail.com> wrote: >> > Hi, >> > >> > This patch fixes https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64387 >> > Which was caused by different in predicates between vec_unpacks_hi >> > and vec_extract_hi. >> >> Why are vec_unpacks_hi_{v8sf,v16sf} expanders different than >> vec_unpacks_hi_v4sf? I think that these should all be expand in the >> same way, similar to vec_unpacks_hi_v4sf. >> > In v4sf case we use movhlps, which is not avalible in v{8,16}sf case.
I see. We are generating vextract<something> here, that has "register_operand" for its operand 1 constraint. Patch is OK. Thanks, Uros.