Hello!

Attached patch fixes all remaining i386 testsuite failures. The header
did say that we can include it also through immintrin.h ...

2011-11-07  Uros Bizjak  <ubiz...@gmail.com>

        * config/i386/f16cintrin: Remove extra _X86INTRIN_H_INCLUDED check.

Tested on x86_64-pc-linux-gnu {,-m32}, committed to mainline SVN.

Uros.
Index: f16cintrin.h
===================================================================
--- f16cintrin.h        (revision 181071)
+++ f16cintrin.h        (working copy)
@@ -1,5 +1,4 @@
-/* Copyright (C) 2011
-   Free Software Foundation, Inc.
+/* Copyright (C) 2011 Free Software Foundation, Inc.
 
    This file is part of GCC.
 
@@ -22,8 +21,7 @@
    see the files COPYING3 and COPYING.RUNTIME respectively.  If not, see
    <http://www.gnu.org/licenses/>.  */
 
-#ifndef _X86INTRIN_H_INCLUDED
-#if (!defined(_X86INTRIN_H_INCLUDED) && !defined(_IMMINTRIN_H_INCLUDED))
+#if !defined _X86INTRIN_H_INCLUDED && !defined _IMMINTRIN_H_INCLUDED
 # error "Never use <f16intrin.h> directly; include <x86intrin.h> or 
<immintrin.h> instead."
 #endif
 
@@ -89,6 +87,6 @@
 #define _mm256_cvtps_ph(A, I) \
   ((__m128i) __builtin_ia32_vcvtps2ph256 ((__v8sf)(__m256) A, (int) (I)))
 #endif /* __OPTIMIZE */
+
 #endif /* _F16CINTRIN_H_INCLUDED */
 #endif /* __F16C__ */
-#endif /* _X86INTRIN_H_INCLUDED */

Reply via email to