[PATCH] D17836: [AArch64] Fixes Store Merge chain in DAG combiner

Weiming Zhao via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 3 16:13:26 PST 2016


weimingz added a comment.

In http://reviews.llvm.org/D17836#367711, @jyknight wrote:

> In http://reviews.llvm.org/D17836#367322, @spatel wrote:
>
> > This is at least related to the problem/patch/discussion in http://reviews.llvm.org/D14834?
>
>
> I believe that patch will fix this issue; I'll test it and get back to you.
>
> Fortunately, Nirav is now (since this week) actually working on finishing the work suggested by the last comments in http://reviews.llvm.org/D14834. He should have something to show for that soon.


Yes, I just verified that it fix my issue. But I got some lit test fails.


Repository:
  rL LLVM

http://reviews.llvm.org/D17836





More information about the llvm-commits mailing list