[Bug ipa/108007] [10/11/12/13 Regression] wrong code at -Os and above with "-fno-dce -fno-tree-dce" on x86_64-linux-gnu

2023-04-03 Thread jakub at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=108007

--- Comment #8 from Jakub Jelinek  ---
E.g. PR93385.

[Bug ipa/108007] [10/11/12/13 Regression] wrong code at -Os and above with "-fno-dce -fno-tree-dce" on x86_64-linux-gnu

2023-04-03 Thread jakub at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=108007

Jakub Jelinek  changed:

   What|Removed |Added

 CC||jakub at gcc dot gnu.org

--- Comment #7 from Jakub Jelinek  ---
Martin, any progress on this?
Feels like PRs we've discussed in the past (and some of them have been fixed),
where IPA passes assume something isn't really needed to be computed because
all its uses will be dead, but doesn't actually DCE them and because DCE is
disabled, it is still needed?

[Bug ipa/108007] [10/11/12/13 Regression] wrong code at -Os and above with "-fno-dce -fno-tree-dce" on x86_64-linux-gnu

2022-12-21 Thread rguenth at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=108007

Richard Biener  changed:

   What|Removed |Added

   Priority|P3  |P2
   Last reconfirmed|2022-12-07 00:00:00 |2022-12-21

--- Comment #6 from Richard Biener  ---
Martin, can you please have a look?

Summary for node main/10:
  Returns value
  No parameter information.

  Summary for edge main/10->k/9:
return value ignored

Summary for node k/9:
  No parameter information.

  Summary for edge k/9->i/8:
return value used only to compute caller return value

Summary for node i/8:
  Returns value
  No parameter information.


the k->i edge info looks misleading (the caller returns nothing), but
the uses are all "dead".  Eventually those dead stmts are now supposed
to be cleaned up by the inliner because of the possibility of -fno-[tree-]dce,
just in this case that isn't working?

Not sure if really P2, but I guess return value removal isn't yet handled
in that code path?

[Bug ipa/108007] [10/11/12/13 Regression] wrong code at -Os and above with "-fno-dce -fno-tree-dce" on x86_64-linux-gnu

2022-12-08 Thread marxin at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=108007

Martin Liška  changed:

   What|Removed |Added

 CC||jamborm at gcc dot gnu.org

--- Comment #5 from Martin Liška  ---
(In reply to Andrew Pinski from comment #4)
> Most likely due to r10-3311-gff6686d2e5f797 (the new IPA-SRA).

Yes, it's since the revision.

[Bug ipa/108007] [10/11/12/13 Regression] wrong code at -Os and above with "-fno-dce -fno-tree-dce" on x86_64-linux-gnu

2022-12-07 Thread pinskia at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=108007

Andrew Pinski  changed:

   What|Removed |Added

   Keywords|needs-bisection |

--- Comment #4 from Andrew Pinski  ---
Most likely due to r10-3311-gff6686d2e5f797 (the new IPA-SRA).

[Bug ipa/108007] [10/11/12/13 Regression] wrong code at -Os and above with "-fno-dce -fno-tree-dce" on x86_64-linux-gnu

2022-12-07 Thread pinskia at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=108007

Andrew Pinski  changed:

   What|Removed |Added

   Last reconfirmed||2022-12-07
 Status|UNCONFIRMED |NEW
   Keywords||needs-bisection
 Ever confirmed|0   |1

--- Comment #3 from Andrew Pinski  ---
Confirmed.

[Bug ipa/108007] [10/11/12/13 Regression] wrong code at -Os and above with "-fno-dce -fno-tree-dce" on x86_64-linux-gnu

2022-12-07 Thread pinskia at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=108007

Andrew Pinski  changed:

   What|Removed |Added

Summary|wrong code at -Os and above |[10/11/12/13 Regression]
   |with "-fno-dce  |wrong code at -Os and above
   |-fno-tree-dce" on   |with "-fno-dce
   |x86_64-linux-gnu|-fno-tree-dce" on
   ||x86_64-linux-gnu
  Known to fail||10.1.0, 11.1.0, 12.2.0
   Target Milestone|--- |10.5
 CC||marxin at gcc dot gnu.org
Version|unknown |13.0
  Component|tree-optimization   |ipa
  Known to work||9.1.0

--- Comment #2 from Andrew Pinski  ---
== IPA-SRA decisions ==

Evaluating analysis results for i/8
  Will remove return value.
  Created new node i.isra/12