[PATCH] Add Load Combine Pass
atrick at apple.com
Wed May 7 12:43:50 PDT 2014
On May 7, 2014, at 12:19 PM, Michael Spencer <bigcheesegs at gmail.com> wrote:
> As for targets that have to split unaligned loads. I tested the original case on ARM and llvm actually generates slightly better code when SDAG splits the combined load than if we just leave it alone. I'll check other targets, but I believe that we do a pretty good job splicing. The only bad cases I ran into were where the combined load isn't actually used combined, it's just directly resplit via shr/trunc. Here I believe we will need to teach SDAG to split these loads back up, but that should be easy (but should be done before this lands).
That sounds great. I’m also fine with the conservative TTI approach, like Hal described.
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the llvm-commits