[Bug libstdc++/113811] std::rotate does 64-bit signed division

2024-02-15 Thread redi at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=113811 Jonathan Wakely changed: What|Removed |Added Resolution|--- |FIXED Status|NEW

[Bug libstdc++/113811] std::rotate does 64-bit signed division

2024-02-15 Thread cvs-commit at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=113811 --- Comment #4 from GCC Commits --- The master branch has been updated by Jonathan Wakely : https://gcc.gnu.org/g:4d819db7f229a23cb15ef68f310e0bb51d201c45 commit r14-9001-g4d819db7f229a23cb15ef68f310e0bb51d201c45 Author: Jonathan Wakely Date:

[Bug libstdc++/113811] std::rotate does 64-bit signed division

2024-02-08 Thread redi at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=113811 --- Comment #3 from Jonathan Wakely --- It seems fairly easy to do: commit 12a028d76bbdf26d34d4d90a2ecdc39c6c0a4bd4 (HEAD -> master) Author: Jonathan Wakely Date: Thu Feb 8 15:40:32 2024 libstdc++: Use unsigned division in std::rotate [P

[Bug libstdc++/113811] std::rotate does 64-bit signed division

2024-02-08 Thread redi at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=113811 Jonathan Wakely changed: What|Removed |Added Status|UNCONFIRMED |NEW Last reconfirmed|

[Bug libstdc++/113811] std::rotate does 64-bit signed division

2024-02-08 Thread rguenth at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=113811 --- Comment #1 from Richard Biener --- In case __n is the minimum signed integer and __k is -1 the division would also trap ;) So yes, they should be unsigned.