Re: [PATCH v2] x86-64: Optimize strcmp/wcscmp with AVX2

[Alexander Monakov <amonakov at ispras dot ru>]

On Mon, 4 Jun 2018, Leonardo Sandoval wrote:
> 
> right, perhaps microbenchmarks does not tell us much on this case
> because AVX and non-AVX is not mixed. Also, if you look at the patch,
> upper ymm bits are cleared (vzeroupper) before returning from strcmp,
> thus there is no perf penalty in storing these and then restoring when
> other AVX code is called again.

Agreed, but I don't understand why you're bringing up the vzeroupper
aspect, my concern was about frequency limits only.

> As I said before, using strcmp wont hurt performance at all (internal
> HW perf team confirmed what I said) because we are not using any opcode
> that that may drop frequency.

Okay. I didn't manage to find confirmations on the Internet though.
In my previous mail I gave a link to an Intel whitepaper that makes
no such indications. Also there's a presentation from CERN saying,

    "Compiling with AVX, or even just using a handful of AVX-256 
     instructions at runtime, will most probably make your program
     globally slower"

(in context of using AVX on Haswell)
URL: https://indico.cern.ch/event/327306/contributions/760669/attachments/635800/875267/HaswellConundrum.pdf

> if you have a test scenario to prove the 5% drop, I would like to  
> test it and discuss it further.

I don't have access to a range of Haswell/Broadwell/Skylake CPUs to test.
If the PDFs I've referenced are in fact incomplete or in error w.r.t.
AVX frequency limits, and you have links to more accurate documents, can
you please share them?

FWIW, on one Haswell CPU I was able to reproduce turbo limits appearing
with non-FMA FP AVX usage, but not INT AVX2. This indicates that on Haswell
the situation is different than what you said initially ("partially true for
AVX2 FMA and AVX512").

Alexander