On Tue, Jun 24, 2025 at 4:54 AM Cui, Lili <lili....@intel.com> wrote:
> > > > From: Lili Cui <lili....@intel.com> > > > > > > > > Hi Uros, > > > > > > > > I need to remove another assertion in the shrink wrap separate patch. > > > Added two cases for changing the CHECK_STACK_LIMIT value. > > > > > > > > The default values for CHECK_STACK_LIMIT for target wingw and option > > > > - > > > mstack-arg-probe are 4000 and (-1) respectively. In this case, shrink > > > wrap separate will go to the branch. This assertion needs to be removed. > > > > > > > > Sergei verified the patch in his environment, and it passed the test. > > > > Bootstrapped & regtested on x86-64-pc-linux-gnu. > > > > > > > > Ok for master? > > > > > > > > Thanks, > > > > Lili. > > > > > > > > > > > > gcc/ChangeLog: > > > > > > > > PR target/120741 > > > > * config/i386/i386.cc (ix86_expand_prologue): > > > > Remove 1 assertions. > > > > > > > > gcc/testsuite/ChangeLog: > > > > > > > > PR target/120741 > > > > * gcc.target/i386/pr120741.c: New test. > > > > * gcc.target/i386/shrink-wrap-separate-mingw.c: Likewise. > > > > > > > > > OK if the tests also pass for i?86-*-*. > Yes, they are. I committed this patch. Thanks, I will follow up with my patch. Uros.
