Hi Gerolf, <div><br></div><div>Why do you want to see the results on x86? SeparateConstOffsetFromGEP is only used by NVPTX and AArch64 so far. I don't think it will affect the compilation time or the compiled code on x86. </div><div><br></div><div>Jingyue<br><br><div class="gmail_quote">On Tue Nov 11 2014 at 10:59:10 AM Gerolf Hoflehner <<a href="mailto:ghoflehner@apple.com">ghoflehner@apple.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Hao,<br>
<br>
could you share your current performance data + options you use? With your patch I see small gains on astar + xalancbmk, but a ~4% loss on libquantum.  The data is based on the 11/07 trunk on top of r221549, O3 + LTO, cyclone, ref input.<br>
In addition to cortex, could you run on x86 also?<br>
<br>
Thanks<br>
Gerolf<br>
<br>
<br>
On Oct 19, 2014, at 11:39 PM, Hao Liu <<a href="mailto:Hao.Liu@arm.com" target="_blank">Hao.Liu@arm.com</a>> wrote:<br>
<br>
> Hi t.p.northover, jingyue,<br>
><br>
> Hi Tim, Jingyue and other reviewers,<br>
><br>
> This patch is based on <a href="http://reviews.llvm.org/D5863" target="_blank">http://reviews.llvm.org/D5863</a>, which fixes some problems in the SeparateConstOffsetFromGEP pass. So please apply that patch first if you want to have a try.<br>
><br>
> We find LLVM cannot handle CSE well in GEPs (getelementptrs). The GEP can be very complex, it can has many mixed constant indices and variable indices. The variable indices may also contain other constants. And such complex GEPs are usually kept in CodeGen. But as CodeGen can only see in one basic block. For the GEPs across basic blocks (e.g, two GEPs used in two different basic blocks or one GEP used in two basic blocks), it may have CSE opportunities and will be missed.<br>
><br>
> Currently there is a pass called SeparateConstOffsetFromGEP, which can separate constant within variable indices and split a complex GEP into two GEPs. But this is not enough for the problem that GEPs across basic blocks I mentioned above.<br>
><br>
> So I improve this pass. It will separate constant within indices for both sequential and struct types. And most important is that it will also transform complex GEPs into a "ptrtoint + arithmetic + inttoptr" form, so that it is able to find CSE opportunities across basic blocks. Also it can benefit address sinking logic in CodeGenPrepare for those complex GEPs which can not be matched by addressing mode, as part of the address calculation can still be sunk. The address sinking logic can result in better addressing mode. EarlyCSE pass is called after this pass to do CSE. LICM pass is also called to do loop invariant code motion in case any of the address calculations are invariant.<br>
><br>
> If we don't find CSE opportunities after such arithmetic  transformation, it still has no obvious regression on performance, as it will always do such transformation in CodeGen. We just move such transformation several passes ahead of CodeGen.<br>
><br>
> I tested the performance for A57 of AArch64 target on SPEC CPU2000 and SPEC CPU2006. It has no obvious regressions and has improvements on following benchmarks:<br>
> spec2006.473.astar     4.7%<br>
> spec2006.444.namd    3.0%<br>
> spec2006.445.gobmk  2.5%<br>
><br>
> For the benchmarks don't have obvious improvement, we can also see the address calculation and addressing mode are better from the assembly code.<br>
><br>
> For other targets like NVPTX, I can not test this patch. I think this patch can also benefit the performance, at least it has no regression.<br>
><br>
> Review please.<br>
><br>
> Thanks,<br>
> -Hao<br>
><br>
> <a href="http://reviews.llvm.org/D5864" target="_blank">http://reviews.llvm.org/D5864</a><br>
><br>
> Files:<br>
>  include/llvm/Transforms/<u></u>Scalar.h<br>
>  lib/Target/AArch64/<u></u>AArch64TargetMachine.cpp<br>
>  lib/Transforms/Scalar/<u></u>SeparateConstOffsetFromGEP.cpp<br>
>  test/CodeGen/AArch64/aarch64-<u></u>gep-opt.ll<br>
>  test/CodeGen/AArch64/arm64-<u></u>addr-mode-folding.ll<br>
>  test/CodeGen/AArch64/arm64-<u></u>cse.ll<br>
> <D5864.15133.patch>___________<u></u>______________________________<u></u>______<br>
> llvm-commits mailing list<br>
> <a href="mailto:llvm-commits@cs.uiuc.edu" target="_blank">llvm-commits@cs.uiuc.edu</a><br>
> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/<u></u>mailman/listinfo/llvm-commits</a><br>
<br>
</blockquote></div></div>