[PATCH] D108699: [LAA] Analyze pointers forked by a select
Graham Hunter via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Apr 12 05:32:27 PDT 2022
huntergr added a comment.
In D108699#3441724 <https://reviews.llvm.org/D108699#3441724>, @fhahn wrote:
> Thanks for the update!
>
> I think it would be good to split this up to have a first patch that just adds very limited support in `findForkedSCEVs` and then gradually add support for more cases separately. This also makes it easier to make sure all code paths in `findForkedSCEVs` are covered by the unit tests.
>
> I went ahead and rebased D114487 <https://reviews.llvm.org/D114487> and stripped the fork analysis to the bare minimum. I'd be happy to land the scaffolding in D114487 <https://reviews.llvm.org/D114487> separately and this patch and following could add the sophisticated fork analysis.
Sure, that sounds like a good plan. I'll try and review your patch this week.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D108699/new/
https://reviews.llvm.org/D108699
More information about the llvm-commits
mailing list