[Bug target/114510] [14 Regression] missed proping of multiply by 2 into address of load/stores

2024-04-03 Thread tnfchris at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=114510

Tamar Christina  changed:

   What|Removed |Added

 CC||tnfchris at gcc dot gnu.org

--- Comment #3 from Tamar Christina  ---
Richard's patch should fix this next year.

https://gcc.gnu.org/pipermail/gcc-patches/2023-October/634166.html

We'll then stop relying on combine or other passes to fix this.

[Bug target/114510] [14 Regression] missed proping of multiply by 2 into address of load/stores

2024-03-29 Thread law at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=114510

Jeffrey A. Law  changed:

   What|Removed |Added

 CC||law at gcc dot gnu.org
   Priority|P3  |P2

[Bug target/114510] [14 Regression] missed proping of multiply by 2 into address of load/stores

2024-03-28 Thread pinskia at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=114510

Andrew Pinski  changed:

   What|Removed |Added

   Target Milestone|--- |14.0

[Bug target/114510] [14 Regression] missed proping of multiply by 2 into address of load/stores

2024-03-28 Thread pinskia at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=114510

--- Comment #2 from Andrew Pinski  ---
Note I was inspired to test this due to LLVM change:
https://github.com/llvm/llvm-project/pull/86894 .

[Bug target/114510] [14 Regression] missed proping of multiply by 2 into address of load/stores

2024-03-28 Thread pinskia at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=114510

--- Comment #1 from Andrew Pinski  ---
Note it looks like fwprop was doing it in GCC 13 but not on the trunk due to a
cost reason. I have not looked further. There could be some cost issues in the
aarch64 backend dealing with address costs too.