subject:"\[Bug debug\/83419\] \[8 Regression\] \-fcompare\-debug failure \(length\)"

[Bug debug/83419] [8 Regression] -fcompare-debug failure (length)

2017-12-21 Thread aoliva at gcc dot gnu.org

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83419 --- Comment #9 from Alexandre Oliva --- Author: aoliva Date: Fri Dec 22 02:07:31 2017 New Revision: 255966 URL: https://gcc.gnu.org/viewcvs?rev=255966=gcc=rev Log: [SFN] sync up debug-only stmt list's side effects with empty stmts too for

[Bug debug/83419] [8 Regression] -fcompare-debug failure (length)

2017-12-21 Thread aoliva at gcc dot gnu.org

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83419 Alexandre Oliva changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution|---

[Bug debug/83419] [8 Regression] -fcompare-debug failure (length)

2017-12-21 Thread aoliva at gcc dot gnu.org

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83419 --- Comment #7 from Alexandre Oliva --- Author: aoliva Date: Thu Dec 21 18:14:06 2017 New Revision: 255947 URL: https://gcc.gnu.org/viewcvs?rev=255947=gcc=rev Log: [SFN] propagate single-nondebug-stmt's side effects to enclosing list

[Bug debug/83419] [8 Regression] -fcompare-debug failure (length)

2017-12-20 Thread aoliva at gcc dot gnu.org

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83419 Alexandre Oliva changed: What|Removed |Added Status|NEW |ASSIGNED

[Bug debug/83419] [8 Regression] -fcompare-debug failure (length)

2017-12-18 Thread jakub at gcc dot gnu.org

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83419 Jakub Jelinek changed: What|Removed |Added Status|ASSIGNED|NEW Assignee|jakub at gcc dot

[Bug debug/83419] [8 Regression] -fcompare-debug failure (length)

2017-12-14 Thread jakub at gcc dot gnu.org

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

[Bug debug/83419] [8 Regression] -fcompare-debug failure (length)

2017-12-14 Thread jakub at gcc dot gnu.org

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83419 Jakub Jelinek changed: What|Removed |Added Status|NEW |ASSIGNED Assignee|unassigned

[Bug debug/83419] [8 Regression] -fcompare-debug failure (length)

2017-12-14 Thread jakub at gcc dot gnu.org

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83419 --- Comment #2 from Jakub Jelinek --- The difference is in shortcut_cond_expr, which does: 3640 bool then_se = then_ && TREE_SIDE_EFFECTS (then_); 3641 bool else_se = else_ && TREE_SIDE_EFFECTS (else_); Now, else_ in one case is: 0;