[PATCH] D71539: [SCEV] Look through trivial PHIs.

Mikael Holmén via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 30 01:02:44 PDT 2021


uabelho added a comment.

In D71539#3030226 <https://reviews.llvm.org/D71539#3030226>, @fhahn wrote:

> In D71539#3029955 <https://reviews.llvm.org/D71539#3029955>, @fhahn wrote:
>
>> @uabelho that looks like a different cause, as DependenceAnalysis is not involved. Taking a look now
>
> Looks like the issue was missing SCEV invalidation after changing the incoming value of a phi in IndVarSimplify. Should be fixed by 0b4a4cc72d81 <https://reviews.llvm.org/rG0b4a4cc72d81a522230a7270abdfa804aa49d89b>

Yep, thanks!


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D71539/new/

https://reviews.llvm.org/D71539



More information about the llvm-commits mailing list