https://gcc.gnu.org/bugzilla/show_bug.cgi?id=120987

Andrew Pinski <pinskia at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |needs-reduction

--- Comment #6 from Andrew Pinski <pinskia at gcc dot gnu.org> ---
(In reply to Tom de Vries from comment #3)
> The investigation in the gdb PR has identified a range of bad commits.
> 
> First bad commit:
> - commit aae723d360c
>   ("sra: SRA of non-escaped aggregates passed by reference to calls")

I suspect this just exposes the latent issue. This pushes for more SRA on some
arguments.

> 
> First good commit after first bad commit:
> - commit cee7d080d5c
>   ("Ignore conditions guarding __builtin_unreachable in inliner metrics")

And this one just make the latent issue go away. This one just changes inlining
heurstics and would not have fixed the latent issue.

Reply via email to