https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86089
Martin Sebor changed:
What|Removed |Added
CC||dcb314 at hotmail dot com
--- Comment
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86089
Martin Liška changed:
What|Removed |Added
Status|ASSIGNED|RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86089
--- Comment #13 from Martin Liška ---
Author: marxin
Date: Mon Jun 11 11:47:00 2018
New Revision: 261434
URL: https://gcc.gnu.org/viewcvs?rev=261434=gcc=rev
Log:
Fix typo done in MPX removal (PR tree-optimization/86089).
2018-06-11 Martin
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86089
--- Comment #12 from Jürgen Reuter ---
(In reply to Martin Liška from comment #11)
> Patch candidate sent here:
> https://gcc.gnu.org/ml/gcc-patches/2018-06/msg00546.html
Thanks for the patch, Martin. I checked that our code works again also
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86089
--- Comment #11 from Martin Liška ---
Patch candidate sent here:
https://gcc.gnu.org/ml/gcc-patches/2018-06/msg00546.html
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86089
--- Comment #10 from Jürgen Reuter ---
I can confirm that re-introducing the line
case BUILT_IN_STRCPY_CHK:
in lines 619/20 in tree-ssa-strlen.c does indeed solve this problem and also
the problem (ICE) with our code reported in PR86103.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86089
--- Comment #9 from Martin Liška ---
*** Bug 86103 has been marked as a duplicate of this bug. ***
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86089
Martin Liška changed:
What|Removed |Added
Status|NEW |ASSIGNED
Assignee|unassigned
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86089
--- Comment #7 from Jürgen Reuter ---
Ah I see, so when Martin Liska removed the MPX-support in r261304 you removed
too much from the tree-ssa-strlen.c file? Then a rather fast fix is hopefully
possible.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86089
Martin Sebor changed:
What|Removed |Added
Status|WAITING |NEW
CC|
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86089
--- Comment #5 from Jürgen Reuter ---
And this is the assembler code when executing -S -O1 (cf. below), while for the
-O2
version the compilation leads to an assembler file showing only the .text line:
.text
.globl _hoo
_hoo:
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86089
--- Comment #4 from Jürgen Reuter ---
This is the dump-tree-original file, which is produced despite of the ICE, and
which is identical for -O1 and -O2:
;; Function __sputc (null)
;; enabled by -tree-original
{
if ( --_p->_w >= 0 || _p->_w
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86089
--- Comment #3 from Jürgen Reuter ---
I can definitely confirm this problem. Works with -O0 and -O1. Fails with -O2.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86089
Jürgen Reuter changed:
What|Removed |Added
CC||juergen.reuter at desy dot de
---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86089
Martin Sebor changed:
What|Removed |Added
Status|UNCONFIRMED |WAITING
Last reconfirmed|
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86089
Andrew Pinski changed:
What|Removed |Added
Keywords||ice-on-valid-code
Component|c
16 matches
Mail list logo