On Wed, Jul 27, 2011 at 9:05 AM, Kirill Yukhin <kirill.yuk...@gmail.com> wrote:
> Thanks for inputs! I'll do it today.
>
> Just ine point.
> How AVX is connected to LZCNT features?
> AVX requires OS support since it has wider registers etc.
> LZCNT need no support from OS side, so from my point of view it is
> redundant to check in lzcnt-check.h presence of AVX support from OS
> side.
> Or I get you wrong?
Ah, I see. I got distracted by the wrong comment in your patch:
+# Return 1 if the target supports running AVX executables, 0 otherwise.
+
+proc check_effective_target_lzcnt_runtime { } {
+ if { [check_effective_target_lzcnt]
+ && [check_lzcnt_hw_available] } {
+ return 1
+ }
+ return 0
+}
(I will add avx-os-support.h myself later today).
Uros.
