[PATCH] D31068: [SDAG] Expand MergedConsecutiveStores to better handle Giving up in Chain Analysis

Clement Courbet via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 23 08:46:14 PDT 2018


courbet added a comment.

In https://reviews.llvm.org/D31068#1267984, @niravd wrote:

> Rebase to head. Some changes to alias make this require a much longer chain of stores. The need is clearly less than when I wrote this patch a year ago, but still has a minor positive effect.


Do you think likely to happen in practice ?


Repository:
  rL LLVM

https://reviews.llvm.org/D31068





More information about the llvm-commits mailing list