Applied to trunk, thanks! --Philipp.
On Tue, 29 Jul 2025 at 15:45, Jeff Law <jeffreya...@gmail.com> wrote: > > > > On 7/28/25 5:00 AM, Konstantinos Eleftheriou wrote: > > While scanning the instructions and upon reaching an instruction that > > doesn't satisfy the constraints that we have set, we were removing the > > already detected stores, but we were continuing adding stores from that > > point onward. This was causing issues when the address ranges from later > > stores overlapped with the load's address, leading to partial and wrong > > update of the register containing the loaded value. > > > > With this patch, we are skipping the tranformation for stores that operate > > on the load's address range, when stores that operate on the same range > > have been deleted due to constraint violations. > > > > PR rtl-optimization/119795 > > > > gcc/ChangeLog: > > > > * avoid-store-forwarding.cc > > (store_forwarding_analyzer::avoid_store_forwarding): Skip > > transformations for stores that operate on the same address > > range as deleted ones. > > > > gcc/testsuite/ChangeLog: > > > > * gcc.target/i386/pr119795.c: New test. > OK. Thanks for correcting the missing conditional. > jeff > > >
