[Bug other/66827] [6 Regression] left shifts of negative value warnings due to C++14 switch
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66827 Jakub Jelinek changed: What|Removed |Added Status|UNCONFIRMED |RESOLVED Resolution|--- |FIXED --- Comment #10 from Jakub Jelinek --- Fixed.
[Bug other/66827] [6 Regression] left shifts of negative value warnings due to C++14 switch
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66827 --- Comment #9 from Nick Clifton --- Author: nickc Date: Mon Dec 21 08:23:35 2015 New Revision: 231873 URL: https://gcc.gnu.org/viewcvs?rev=231873=gcc=rev Log: PR 66827 * regex.c (EXTRACT_NUMBER): Cast sign byte to unsigned before left shifting. Modified: trunk/libiberty/ChangeLog trunk/libiberty/regex.c
[Bug other/66827] [6 Regression] left shifts of negative value warnings due to C++14 switch
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66827 Nick Clifton changed: What|Removed |Added CC||nickc at redhat dot com --- Comment #8 from Nick Clifton --- Created attachment 37079 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=37079=edit Convert to unsigned before shifting (In reply to Jakub Jelinek from comment #7) > The question is about the others, but it has been so long that the > locations are all useless. I have just checked. The only problems that still remain are in libiberty/regex.c. The uploaded patch fixes this problem. I plan to submit it for review after running a few more regression tests.
[Bug other/66827] [6 Regression] left shifts of negative value warnings due to C++14 switch
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66827 Jakub Jelinek changed: What|Removed |Added CC||jakub at gcc dot gnu.org --- Comment #7 from Jakub Jelinek --- I believe UNKNOWN_DEP_COST stuff has been fixed in the meantime. The question is about the others, but it has been so long that the locations are all useless.
[Bug other/66827] [6 Regression] left shifts of negative value warnings due to C++14 switch
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66827 --- Comment #6 from Paolo Bonzini --- If you really want to fix it, (-(1 << 19)) is the best. The real fix would be to lobby the C/C++ committees so that left shift of a negative value is unspecified behavior rather than undefined.
[Bug other/66827] [6 Regression] left shifts of negative value warnings due to C++14 switch
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66827 Paolo Bonzini changed: What|Removed |Added CC||bonzini at gnu dot org --- Comment #4 from Paolo Bonzini --- Using #define UNKNOWN_DEP_COST (-1u<<19) is dangerous in case UNKNOWN_DEP_COST is later assigned to a long. Do not do this please.
[Bug other/66827] [6 Regression] left shifts of negative value warnings due to C++14 switch
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66827 --- Comment #5 from Vittorio Zecca --- I am traveling now so I cannot double check your hint. What do you suggest?
[Bug other/66827] [6 Regression] left shifts of negative value warnings due to C++14 switch
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66827 --- Comment #3 from Vittorio Zecca --- (In reply to Mikhail Maltsev from comment #1) > gcc/haifa-sched.c:1164:24 > gcc/haifa-sched.c:1442:26 > gcc/sched-deps.c:112:20 > > are caused by the following macro definition in gcc/sched-int.h:243: > #define UNKNOWN_DEP_COST (-1<<19) I fixed this one with #define UNKNOWN_DEP_COST (-1u<<19)
[Bug other/66827] [6 Regression] left shifts of negative value warnings due to C++14 switch
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66827 Vittorio Zecca zeccav at gmail dot com changed: What|Removed |Added CC||zeccav at gmail dot com --- Comment #2 from Vittorio Zecca zeccav at gmail dot com --- The warnings in regex.c are already described in issue 64920
[Bug other/66827] [6 Regression] left shifts of negative value warnings due to C++14 switch
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66827 Mikhail Maltsev miyuki at gcc dot gnu.org changed: What|Removed |Added CC||miyuki at gcc dot gnu.org --- Comment #1 from Mikhail Maltsev miyuki at gcc dot gnu.org --- gcc/haifa-sched.c:1164:24 gcc/haifa-sched.c:1442:26 gcc/sched-deps.c:112:20 are caused by the following macro definition in gcc/sched-int.h:243: #define UNKNOWN_DEP_COST (-119)
[Bug other/66827] [6 Regression] left shifts of negative value warnings due to C++14 switch
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66827 Richard Biener rguenth at gcc dot gnu.org changed: What|Removed |Added Target Milestone|--- |6.0