Andrew Carlotti <andrew.carlo...@arm.com> writes:
> This test started failing on aarch64 after 0cfc9c95 in 2023 ("Phi
> analyzer - Initialize with range instead of a tree.").
>
> The only change visible in the pass dumps prior to thread2 is the upper
> bounds of some ranges are reduced from +INF to 7, consistent with the
> bitamsk information. After thread2, there are changes in the control
> flow, but only affecting edges that are obviously never taken (from
> basic blocks 6 through 12). These are cleaned up in the following pass,
> but the final codegen remains different.
>
> There isn't anything obviously wrong with the change in dump output, so
> let's just update the test expectations (as has happened previously
> here).
>
> I think this is an obvious patch, but I'll wait a few days before
> committing.
Yeah, these tests are finicky to say the least.
The change LGTM. Thanks for your analysis here.
Aldy
