public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
* Re: [Patch] PR71017 - libgcc/config/i386/cpuinfo.c:346:17: runtime error: left shift of 1 by 31 places cannot be represented in type 'int'
@ 2017-01-09 19:37 Uros Bizjak
  2017-01-09 22:05 ` Dominique d'Humières
  0 siblings, 1 reply; 4+ messages in thread
From: Uros Bizjak @ 2017-01-09 19:37 UTC (permalink / raw)
  To: gcc-patches; +Cc: Dominique Dhumieres, kyukhin

Hello!

> The following patch fixes errors of the kind
>
> libgcc/config/i386/cpuinfo.c:260:17: runtime error: left shift of 1 by 31 places cannot be
>  represented in type ‘int'
>
> 2017-01-07  Dominique d'Humieres  <dominiq@lps.ens.fr>
>
>         PR target/71017
>         * config/i386/cpuid.h: Fix undefined behavior.

> Is it OK for trunk/branches?

OK.

Thanks,
Uros.

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [Patch] PR71017 - libgcc/config/i386/cpuinfo.c:346:17: runtime error: left shift of 1 by 31 places cannot be represented in type 'int'
  2017-01-09 19:37 [Patch] PR71017 - libgcc/config/i386/cpuinfo.c:346:17: runtime error: left shift of 1 by 31 places cannot be represented in type 'int' Uros Bizjak
@ 2017-01-09 22:05 ` Dominique d'Humières
       [not found]   ` <CAFULd4Ze4+MTHWPEGq+cKMFmdmVAXy_xzCkAvcpg9XEwTxPHOQ@mail.gmail.com>
  0 siblings, 1 reply; 4+ messages in thread
From: Dominique d'Humières @ 2017-01-09 22:05 UTC (permalink / raw)
  To: Uros Bizjak; +Cc: gcc-patches, kyukhin


> Le 9 janv. 2017 à 20:37, Uros Bizjak <ubizjak@gmail.com> a écrit :
> 
> Hello!
> 
>> The following patch fixes errors of the kind
>> 
>> libgcc/config/i386/cpuinfo.c:260:17: runtime error: left shift of 1 by 31 places cannot be
>> represented in type ‘int'
>> 
>> 2017-01-07  Dominique d'Humieres  <dominiq@lps.ens.fr>
>> 
>>        PR target/71017
>>        * config/i386/cpuid.h: Fix undefined behavior.
> 
>> Is it OK for trunk/branches?
> 
> OK.
> 
> Thanks,
> Uros.

Thanks, committed on the trunk as revision r244248. Is it OK for the 5 and 6 branches?

Dominique

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Fwd: [Patch] PR71017 - libgcc/config/i386/cpuinfo.c:346:17: runtime error: left shift of 1 by 31 places cannot be represented in type 'int'
       [not found]   ` <CAFULd4Ze4+MTHWPEGq+cKMFmdmVAXy_xzCkAvcpg9XEwTxPHOQ@mail.gmail.com>
@ 2017-01-10  8:13     ` Uros Bizjak
  0 siblings, 0 replies; 4+ messages in thread
From: Uros Bizjak @ 2017-01-10  8:13 UTC (permalink / raw)
  To: gcc-patches; +Cc: Dominique Dhumieres, Kirill Yukhin

On Mon, Jan 9, 2017 at 11:05 PM, Dominique d'Humières
<dominiq@lps.ens.fr> wrote:

>
> > Le 9 janv. 2017 à 20:37, Uros Bizjak <ubizjak@gmail.com> a écrit :
> >
> > Hello!
> >
> >> The following patch fixes errors of the kind
> >>
> >> libgcc/config/i386/cpuinfo.c:260:17: runtime error: left shift of 1 by 31 places cannot be
> >> represented in type ‘int'
> >>
> >> 2017-01-07  Dominique d'Humieres  <dominiq@lps.ens.fr>
> >>
> >>        PR target/71017
> >>        * config/i386/cpuid.h: Fix undefined behavior.
> >
> >> Is it OK for trunk/branches?
>
>
> Thanks, committed on the trunk as revision r244248. Is it OK for the 5 and 6 branches?

Yes, but please wait a couple of days if any problem arises with patch
in mainline.

Uros.

^ permalink raw reply	[flat|nested] 4+ messages in thread

* [Patch] PR71017 - libgcc/config/i386/cpuinfo.c:346:17: runtime error: left shift of 1 by 31 places cannot be represented in type 'int'
@ 2017-01-07 13:14 Dominique d'Humières
  0 siblings, 0 replies; 4+ messages in thread
From: Dominique d'Humières @ 2017-01-07 13:14 UTC (permalink / raw)
  To: GCC-Patches-ML; +Cc: kyukhin

The following patch fixes errors of the kind

libgcc/config/i386/cpuinfo.c:260:17: runtime error: left shift of 1 by 31 places cannot be represented in type ‘int'

2017-01-07  Dominique d'Humieres  <dominiq@lps.ens.fr>

        PR target/71017
        * config/i386/cpuid.h: Fix undefined behavior.

--- ../_clean/gcc/config/i386/cpuid.h	2017-01-01 17:39:04.000000000 +0100
+++ gcc/config/i386/cpuid.h	2017-01-05 15:22:31.000000000 +0100
@@ -91,7 +91,7 @@
 #define bit_AVX512CD	(1 << 28)
 #define bit_SHA		(1 << 29)
 #define bit_AVX512BW	(1 << 30)
-#define bit_AVX512VL	(1 << 31)
+#define bit_AVX512VL	(1u << 31)
 
 /* %ecx */
 #define bit_PREFETCHWT1	  (1 << 0)

Is it OK for trunk/branches?

TIA

Dominique

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2017-01-10  8:13 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-01-09 19:37 [Patch] PR71017 - libgcc/config/i386/cpuinfo.c:346:17: runtime error: left shift of 1 by 31 places cannot be represented in type 'int' Uros Bizjak
2017-01-09 22:05 ` Dominique d'Humières
     [not found]   ` <CAFULd4Ze4+MTHWPEGq+cKMFmdmVAXy_xzCkAvcpg9XEwTxPHOQ@mail.gmail.com>
2017-01-10  8:13     ` Fwd: " Uros Bizjak
  -- strict thread matches above, loose matches on Subject: below --
2017-01-07 13:14 Dominique d'Humières

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).