[LLVMdev] [PATCH] Detect Haswell subarchitecture (i.e. using -march=native)

[Adam Strzelecki]

>> +  bool HasAVX2 = HasAVX && !GetX86CpuIDAndInfo(0x7, &EAX, &EBX, &ECX, &EDX) &&
>> +                 (EBX & 0x20);
> 
> I don't think this guarantees %ecx is 0, does it? Wasn't that the
> entire reason the original code went wrong?

I don’t remember really, but presuming the conclusions of the discussion, seems it is fixed now. It was something about registers when using inline assembly. Anyway this works just fine on all my Haswell machines.

Regards,
-- 
Adam