This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH] Vzeroupper placement/47440


On Sun, Nov 4, 2012 at 2:29 PM, Uros Bizjak <ubizjak@gmail.com> wrote:

> -/* { dg-final { scan-assembler-not "avx_vzeroupper" } } */
> +/* { dg-final { scan-assembler-times "avx_vzeroupper" 3 } } */
>
> (... and a couple of similar testsuite changes ...)
>
> These asm scans were put there for a reason. I assume you have looked
> at these differences and are correct (this also implies that current
> vzeroupper placement code is not optimal or even wrong).

Ah, these extra instructions were inserted with _mm265_* intrinsics.
We decided some time ago, that these should remain, and no attempt to
"optimize" them will be performed. OTOH, automatic insertion won't
emit extra vzeroupper in this case.

So, all is OK.

Thanks,
Uros.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]