subject:"\[Bug middle\-end\/64182\] \[5 Regression\] wide\-int rounding division is broken"

[Bug middle-end/64182] [5 Regression] wide-int rounding division is broken

2014-12-17 Thread mpolacek at gcc dot gnu.org

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64182 Marek Polacek changed: What|Removed |Added CC||mpolacek at gcc dot gnu.org --- Comment

[Bug middle-end/64182] [5 Regression] wide-int rounding division is broken

2014-12-12 Thread rsandifo at gcc dot gnu.org

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64182 --- Comment #9 from rsandifo at gcc dot gnu.org --- Author: rsandifo Date: Fri Dec 12 15:46:57 2014 New Revision: 218678 URL: https://gcc.gnu.org/viewcvs?rev=218678&root=gcc&view=rev Log: gcc/ PR middle-end/64182 * wide-int.h (wi::div_r

[Bug middle-end/64182] [5 Regression] wide-int rounding division is broken

2014-12-11 Thread rsandifo at gcc dot gnu.org

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64182 --- Comment #8 from rsandifo at gcc dot gnu.org --- (In reply to Kenneth Zadeck from comment #7) > I do believe that it is possible to test the double int code if you wrote a > program that used fixed point math.I have never used fixed point

[Bug middle-end/64182] [5 Regression] wide-int rounding division is broken

2014-12-11 Thread zadeck at naturalbridge dot com

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64182 Kenneth Zadeck changed: What|Removed |Added CC||zadeck at naturalbridge dot com --- Com

[Bug middle-end/64182] [5 Regression] wide-int rounding division is broken

2014-12-10 Thread rsandifo at gcc dot gnu.org

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64182 --- Comment #6 from rsandifo at gcc dot gnu.org --- (In reply to Jakub Jelinek from comment #5) > Makes sense. Will you also fix double-int.c when you are on this? OK, I'll take the same approach there. I don't know how to test that code "pro

[Bug middle-end/64182] [5 Regression] wide-int rounding division is broken

2014-12-10 Thread jakub at gcc dot gnu.org

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64182 --- Comment #5 from Jakub Jelinek --- Makes sense. Will you also fix double-int.c when you are on this?

[Bug middle-end/64182] [5 Regression] wide-int rounding division is broken

2014-12-10 Thread rsandifo at gcc dot gnu.org

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64182 --- Comment #4 from rsandifo at gcc dot gnu.org --- (In reply to rsand...@gcc.gnu.org from comment #3) > (In reply to Jakub Jelinek from comment #1) > > Created attachment 34222 [details] > > gcc5-pr64182.patch > > > > So like this (completely

[Bug middle-end/64182] [5 Regression] wide-int rounding division is broken

2014-12-10 Thread rsandifo at gcc dot gnu.org

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64182 rsandifo at gcc dot gnu.org changed: What|Removed |Added Status|UNCONFIRMED |ASSIGNED Last reconfirme

[Bug middle-end/64182] [5 Regression] wide-int rounding division is broken

2014-12-09 Thread jakub at gcc dot gnu.org

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64182 --- Comment #2 from Jakub Jelinek --- Note that double-int implementation looks broken too, it doesn't consider uns (negates both anyway if they are "negative"), and it probably doesn't handle the case of too big remainder either.