[Bug c/59708] clang-compatible checked arithmetic builtins

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=59708 --- Comment #31 from Pat Haugen --- (In reply to Pat Haugen from comment #30) > builtin-arith-overflow-10/11 are target problem, I'm working on a fix. builtin-arith-overflow-10/11 are fixed with the patch for pr64358.

[Bug c/59708] clang-compatible checked arithmetic builtins

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=59708 --- Comment #30 from Pat Haugen --- (In reply to Pat Haugen from comment #29) > (In reply to Jakub Jelinek from comment #28) > > Assuming fixed. > > builtin-arith-overflow-14/17 are fixed with the patch, but > builtin-arith-overflow-1/10/11 stil

[Bug c/59708] clang-compatible checked arithmetic builtins

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=59708 --- Comment #29 from Pat Haugen --- (In reply to Jakub Jelinek from comment #28) > Assuming fixed. builtin-arith-overflow-14/17 are fixed with the patch, but builtin-arith-overflow-1/10/11 still fail for me. [pthaugen@igoo trunk_work]$ grep ov

[Bug c/59708] clang-compatible checked arithmetic builtins

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=59708 Jakub Jelinek changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution|---

[Bug c/59708] clang-compatible checked arithmetic builtins

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=59708 --- Comment #27 from Jakub Jelinek --- Author: jakub Date: Wed Dec 3 20:38:44 2014 New Revision: 218333 URL: https://gcc.gnu.org/viewcvs?rev=218333&root=gcc&view=rev Log: PR c/59708 * expmed.c (expand_widening_mult): Return const0_rtx i

[Bug c/59708] clang-compatible checked arithmetic builtins

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=59708 --- Comment #26 from Jakub Jelinek --- Author: jakub Date: Wed Dec 3 20:35:08 2014 New Revision: 218331 URL: https://gcc.gnu.org/viewcvs?rev=218331&root=gcc&view=rev Log: PR c/59708 * expmed.c (expand_widening_mult): Return const0_rtx i

[Bug c/59708] clang-compatible checked arithmetic builtins

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=59708 --- Comment #25 from Jakub Jelinek --- Created attachment 34179 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=34179&action=edit gcc5-pr59708.patch Untested fix for the ppc32 builtin-arith-overflow-14.c failure. expand_widening_mult somet

[Bug c/59708] clang-compatible checked arithmetic builtins

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=59708 --- Comment #24 from Andreas Schwab --- builtin-arith-overflow-14.c still fails on ppc32 as of r218249.

[Bug c/59708] clang-compatible checked arithmetic builtins

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=59708 --- Comment #23 from Markus Trippelsdorf --- (In reply to Marek Polacek from comment #22) > So fixed? Not on ppc64: trippels@gcc2-power8 ~ % cat out_check15 | grep overflow FAIL: gcc.dg/builtin-arith-overflow-1.c execution test FAIL: c-c++-comm

[Bug c/59708] clang-compatible checked arithmetic builtins

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

[Bug c/59708] clang-compatible checked arithmetic builtins

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=59708 --- Comment #21 from dave.anglin at bell dot net --- On 22-Nov-14, at 2:31 AM, jakub at gcc dot gnu.org wrote: > Is that with r217946 or later? This is now fixed on hppa-unknown-linux-gnu. -- John David Anglindave.ang...@bell.net

[Bug c/59708] clang-compatible checked arithmetic builtins

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=59708 --- Comment #20 from dave.anglin at bell dot net --- On 22-Nov-14, at 2:31 AM, jakub at gcc dot gnu.org wrote: > Is that with r217946 or later? No. My latest build is r217898. -- John David Anglindave.ang...@bell.net

[Bug c/59708] clang-compatible checked arithmetic builtins

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=59708 --- Comment #19 from Jakub Jelinek --- (In reply to John David Anglin from comment #18) > On hppa-unknown-linux-gnu, I get: > > FAIL: c-c++-common/torture/builtin-arith-overflow-14.c -O0 execution test > FAIL: c-c++-common/torture/builtin-ari

[Bug c/59708] clang-compatible checked arithmetic builtins

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=59708 John David Anglin changed: What|Removed |Added CC||danglin at gcc dot gnu.org --- Comme

[Bug c/59708] clang-compatible checked arithmetic builtins

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=59708 --- Comment #17 from Uroš Bizjak --- On alpha-linux-gnu, I got following g++ failures: FAIL: c-c++-common/torture/builtin-arith-overflow-17.c -O0 execution test FAIL: c-c++-common/torture/builtin-arith-overflow-17.c -O2 execution test FAIL

[Bug c/59708] clang-compatible checked arithmetic builtins

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=59708 Markus Trippelsdorf changed: What|Removed |Added CC||trippels at gcc dot gnu.org --- Co

[Bug c/59708] clang-compatible checked arithmetic builtins

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=59708 --- Comment #15 from Jakub Jelinek --- Author: jakub Date: Wed Nov 12 12:28:06 2014 New Revision: 217415 URL: https://gcc.gnu.org/viewcvs?rev=217415&root=gcc&view=rev Log: PR c/59708 * builtin-attrs.def (ATTR_NOTHROW_TYPEGENERIC_LEAF): N

[Bug c/59708] clang-compatible checked arithmetic builtins

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=59708 Jakub Jelinek changed: What|Removed |Added Attachment #33912|0 |1 is obsolete|

[Bug c/59708] clang-compatible checked arithmetic builtins

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=59708 Jakub Jelinek changed: What|Removed |Added Attachment #33891|0 |1 is obsolete|

[Bug c/59708] clang-compatible checked arithmetic builtins

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=59708 Jakub Jelinek changed: What|Removed |Added Attachment #33855|0 |1 is obsolete|

[Bug c/59708] clang-compatible checked arithmetic builtins

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=59708 --- Comment #11 from Jakub Jelinek --- Created attachment 33855 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=33855&action=edit gcc5-pr59708-wip1.patch WIP untested patch, which implements so far the clang compatible builtins (with the ex

[Bug c/59708] clang-compatible checked arithmetic builtins

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=59708 Manuel López-Ibáñez changed: What|Removed |Added CC||noloader at gmail dot com --- Comm

[Bug c/59708] clang-compatible checked arithmetic builtins

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=59708 --- Comment #9 from Martin von Gagern --- (In reply to Andrew Pinski from comment #8) > I rather not have builtins as you can write portable C code that detects > overflow just fine. GCC could have internal functions if needed which are > used w

[Bug c/59708] clang-compatible checked arithmetic builtins

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=59708 --- Comment #8 from Andrew Pinski --- I rather not have builtins as you can write portable C code that detects overflow just fine. GCC could have internal functions if needed which are used when converting the pattern for detecting overflow.

[Bug c/59708] clang-compatible checked arithmetic builtins

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=59708 Martin von Gagern changed: What|Removed |Added CC||Martin.vGagern at gmx dot net --- Co

[Bug c/59708] clang-compatible checked arithmetic builtins

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59708 --- Comment #6 from Josh Triplett --- (In reply to jos...@codesourcery.com from comment #5) > See what I said in about > the issues with something type-generic regarding detecting overflow on > t

[Bug c/59708] clang-compatible checked arithmetic builtins

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59708 --- Comment #5 from joseph at codesourcery dot com --- See what I said in about the issues with something type-generic regarding detecting overflow on types smaller than int.

[Bug c/59708] clang-compatible checked arithmetic builtins

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59708 --- Comment #4 from Josh Triplett --- (In reply to Jakub Jelinek from comment #2) > Not sure I like their naming though, I'd say they should be type generic > builtins handled in the FEs depending on the first argument's type, so that > it works e

[Bug c/59708] clang-compatible checked arithmetic builtins

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59708 --- Comment #3 from Richard Biener --- (In reply to Jakub Jelinek from comment #2) > Not sure I like their naming though, I'd say they should be type generic > builtins handled in the FEs depending on the first argument's type, so that > it works

[Bug c/59708] clang-compatible checked arithmetic builtins

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59708 Jakub Jelinek changed: What|Removed |Added CC||jakub at gcc dot gnu.org --- Comment #2 f

[Bug c/59708] clang-compatible checked arithmetic builtins

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59708 Richard Biener changed: What|Removed |Added Status|UNCONFIRMED |NEW Last reconfirmed|