[Bug rtl-optimization/98973] [11 regression] Wrong code with gcse store motion pass

2021-04-13 Thread law at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98973 --- Comment #12 from Jeffrey A. Law --- IIRC LSM is quite restricted in the types of MEM expressions it will optimize. In particular I think they have to be SYMBOL_REFs which severely limits LSM's effectiveness. I would support removing it give

[Bug rtl-optimization/98973] [11 regression] Wrong code with gcse store motion pass

2021-04-12 Thread ebotcazou at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98973 Eric Botcazou changed: What|Removed |Added Status|NEW |WAITING --- Comment #11 from Eric Botcaz

[Bug rtl-optimization/98973] [11 regression] Wrong code with gcse store motion pass

2021-04-09 Thread rguenth at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98973 Richard Biener changed: What|Removed |Added CC||ebotcazou at gcc dot gnu.org,

[Bug rtl-optimization/98973] [11 regression] Wrong code with gcse store motion pass

2021-02-26 Thread rguenth at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98973 --- Comment #9 from Richard Biener --- Btw, this is likely a latent issue, the store-motion code didn't change since ages and the bisected rev. likely just changed the input to the pass. So the dump difference is -(note 76 75 217 19 [bb 19] NOT

[Bug rtl-optimization/98973] [11 regression] Wrong code with gcse store motion pass

2021-02-05 Thread rguenth at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98973 --- Comment #8 from Richard Biener --- (In reply to Richard Biener from comment #6) > Btw, -fgcse-sm is nowhere enabled by default (same applies to -fgcse-las), > we should consider removing these optimizations (though -fgcse-las at least > sound

[Bug rtl-optimization/98973] [11 regression] Wrong code with gcse store motion pass

2021-02-05 Thread jakub at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98973 Jakub Jelinek changed: What|Removed |Added CC||jakub at gcc dot gnu.org --- Comment #7

[Bug rtl-optimization/98973] [11 regression] Wrong code with gcse store motion pass

2021-02-05 Thread rguenth at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98973 --- Comment #6 from Richard Biener --- Btw, -fgcse-sm is nowhere enabled by default (same applies to -fgcse-las), we should consider removing these optimizations (though -fgcse-las at least sounds useful and I wonder why it is not enabled). GCSE

[Bug rtl-optimization/98973] [11 regression] Wrong code with gcse store motion pass

2021-02-05 Thread krebbel at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98973 --- Comment #5 from Andreas Krebbel --- Created attachment 50132 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=50132&action=edit RTL dump from store motion pass

[Bug rtl-optimization/98973] [11 regression] Wrong code with gcse store motion pass

2021-02-05 Thread krebbel at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98973 --- Comment #4 from Andreas Krebbel --- The update of global variable c is moved out of the loop. Due to that c stays at 8 although it should be counted down to 2.

[Bug rtl-optimization/98973] [11 regression] Wrong code with gcse store motion pass

2021-02-05 Thread krebbel at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98973 --- Comment #3 from Andreas Krebbel --- Created attachment 50131 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=50131&action=edit RTL GCSE dump without -fgcse-sm

[Bug rtl-optimization/98973] [11 regression] Wrong code with gcse store motion pass

2021-02-05 Thread krebbel at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98973 --- Comment #2 from Andreas Krebbel --- Created attachment 50130 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=50130&action=edit RTL GCSE dump with -fgcse-sm

[Bug rtl-optimization/98973] [11 regression] Wrong code with gcse store motion pass

2021-02-05 Thread rguenth at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98973 Richard Biener changed: What|Removed |Added Known to fail||11.0 Keywords|