[Bug middle-end/61903] signed integer overflow in expmed.c store_fixed_bit_filed_1

[mpolacek at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61903

--- Comment #3 from Marek Polacek mpolacek at gcc dot gnu.org ---
Author: mpolacek
Date: Mon Sep  1 09:18:43 2014
New Revision: 214781

URL: https://gcc.gnu.org/viewcvs?rev=214781root=gccview=rev
Log:
PR middle-end/61903
* expmed.c (store_fixed_bit_field_1): Shift UHWI 1 instead of HWI 1.
Change the type of V to unsigned HOST_WIDE_INT.

Modified:
trunk/gcc/ChangeLog
trunk/gcc/expmed.c

[Bug middle-end/61903] signed integer overflow in expmed.c store_fixed_bit_filed_1

[mpolacek at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61903

Marek Polacek mpolacek at gcc dot gnu.org changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 CC||mpolacek at gcc dot gnu.org
 Resolution|--- |FIXED
   Assignee|unassigned at gcc dot gnu.org  |mpolacek at gcc dot 
gnu.org
   Target Milestone|--- |5.0

--- Comment #4 from Marek Polacek mpolacek at gcc dot gnu.org ---
Fixed.

[Bug middle-end/61903] signed integer overflow in expmed.c store_fixed_bit_filed_1

[pinskia at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61903

Andrew Pinski pinskia at gcc dot gnu.org changed:

   What|Removed |Added

 Status|UNCONFIRMED |NEW
   Last reconfirmed||2014-07-25
  Component|c   |middle-end
 Ever confirmed|0   |1

--- Comment #2 from Andrew Pinski pinskia at gcc dot gnu.org ---
  if (bitsize  HOST_BITS_PER_WIDE_INT)
v = ((HOST_WIDE_INT) 1  bitsize) - 1;

Should be an easy fix to change HOST_WIDE_INT to unsigned HOST_WIDE_INT.