This is the mail archive of the gcc@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: Unused variable in avx512fintrin.h


On Tue, Aug 09, 2016 at 03:13:30PM -0700, David Wohlferd wrote:
> I'm looking at gcc/config/i386/avx512fintrin.h, and I see this:
> 
> extern __inline __m256i
> __attribute__ ((__gnu_inline__, __always_inline__, __artificial__))
> _mm512_cvtsepi64_epi32 (__m512i __A)
> {
>   __v8si __O;
>   return (__m256i) __builtin_ia32_pmovsqd512_mask ((__v8di) __A,
>                            (__v8si)
>                            _mm256_undefined_si256 (),
>                            (__mmask8) -1);
> }
> 
> Since r208793, __O appears to be unused?
> 
> For someone with checkin permissions (ie not me), removing this seems like
> an "as obvious."

Indeed, I've committed following to trunk:

Index: ChangeLog
===================================================================
--- ChangeLog	(revision 239309)
+++ ChangeLog	(working copy)
@@ -1,3 +1,8 @@
+2016-08-09  David Wohlferd  <dw@LimeGreenSocks.com>
+
+	* config/i3836/avx512fintrin.h (_mm512_cvtsepi64_epi32): Remove
+	unused variable __O.
+
 2016-08-09  Martin Liska  <mliska@suse.cz>
 
 	* doc/gcov.texi: Change _gcov_dump to __gcov_dump and
Index: config/i386/avx512fintrin.h
===================================================================
--- config/i386/avx512fintrin.h	(revision 239309)
+++ config/i386/avx512fintrin.h	(working copy)
@@ -4903,7 +4903,6 @@ extern __inline __m256i
 __attribute__ ((__gnu_inline__, __always_inline__, __artificial__))
 _mm512_cvtsepi64_epi32 (__m512i __A)
 {
-  __v8si __O;
   return (__m256i) __builtin_ia32_pmovsqd512_mask ((__v8di) __A,
 						   (__v8si)
 						   _mm256_undefined_si256 (),

	Jakub


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