[PATCH] D25914: Redo store splitting in CodeGenPrepare

Wei Mi via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 2 15:07:17 PDT 2016


Chandler, could you take a look? It doesn't have much change compared
with the original version in dag combiner.


On Mon, Oct 24, 2016 at 10:10 AM, Wei Mi <wmi at google.com> wrote:
> wmi created this revision.
> wmi added a reviewer: chandlerc.
> wmi added subscribers: llvm-commits, davidxl, majnemer, arsenm.
> wmi set the repository for this revision to rL LLVM.
>
> This is a succeeding patch for https://reviews.llvm.org/D22840 to address the issue when a value to be merged into an int64 pair is in a different BB. The issue was originally described here: http://lists.llvm.org/pipermail/llvm-commits/Week-of-Mon-20160912/390582.html (Sorry to delay the patch for a long time, because I took a long vacation).
>
> The fix is to redo the store splitting in CodeGenPrepare, so we can match the pattern across multiple BBs or move some instructions into the same BB.
>
> The patch changing the target query interface was put separately here: https://reviews.llvm.org/D24707
>
>
> Repository:
>   rL LLVM
>
> https://reviews.llvm.org/D25914
>
> Files:
>   lib/CodeGen/CodeGenPrepare.cpp
>   test/CodeGen/X86/split-store.ll
>


More information about the llvm-commits mailing list