<div dir="ltr">I tried -Os and effect of new approach significantly increases.<div>I run regular DSE and immediately myDSE. With -Os myDSE removes more than 50% of DSE number.</div><div>Which is expected as -Os inlines less and regular DSE can't remove over function call.</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Apr 16, 2019 at 7:11 AM Alexander Potapenko <<a href="mailto:glider@google.com">glider@google.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Mon, Apr 15, 2019 at 11:02 PM Amara Emerson via llvm-dev<br>
<<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a>> wrote:<br>
><br>
><br>
> > On Apr 15, 2019, at 1:51 PM, Vitaly Buka via llvm-dev <<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a>> wrote:<br>
> ><br>
> > Hi JF,<br>
> ><br>
> >    I've heard that you are interested DSE improvements and maybe we need to be in sync.<br>
> >    So far I experimented with following DSE improvements:<br>
> ><br>
> > * Cross-block DSE, it eliminates additional 7% stores comparing to existing DSE. But it's not visible on benchmarks.<br>
> I take it you couldn’t see any runtime impact? If there’s code size improvements that could also be useful, CTMark in the llvm test suite is a useful subset of benchmarks to check this on (as a baseline use -Os to compare code size).<br>
><br>
> Thanks,<br>
> Amara<br>
> ><br>
> > * Cross-block + Interprocedural analysis to annotate each function argument with:<br>
> >   - can read before write<br>
> >   - will always write<br>
> > This annotations gets me 20% stores deleted additional to the current DSE.<br>
I believe we can only benefit from removing extra stores.<br>
Hot functions in existing benchmarks are probably optimized good<br>
enough already, but speeding up the long tail is also important.<br>
Also, at least the repro in<br>
<a href="https://bugs.llvm.org/show_bug.cgi?id=40527" rel="noreferrer" target="_blank">https://bugs.llvm.org/show_bug.cgi?id=40527</a> has been extracted from a<br>
real kernel benchmark (hackbench), where this extra store costed us<br>
0.45%<br>
<br>
> > This is on LLVM codebase with -ftrivial-auto-var-init=patter.<br>
> ><br>
> > As-is it's less than I expected, so I would like to find good benchmark to decide if we should work to make production code from my experiment.<br>
> ><br>
> > So now I am also planing to try to extend that to whole program analysis.<br>
> > I will cleanup my code and upload this during this weak, if anyone wants to try.<br>
> ><br>
> > Vitaly.<br>
> > _______________________________________________<br>
> > LLVM Developers mailing list<br>
> > <a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a><br>
> > <a href="https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev" rel="noreferrer" target="_blank">https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev</a><br>
><br>
> _______________________________________________<br>
> LLVM Developers mailing list<br>
> <a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a><br>
> <a href="https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev" rel="noreferrer" target="_blank">https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev</a><br>
<br>
<br>
<br>
-- <br>
Alexander Potapenko<br>
Software Engineer<br>
<br>
Google Germany GmbH<br>
Erika-Mann-Straße, 33<br>
80636 München<br>
<br>
Geschäftsführer: Paul Manicle, Halimah DeLaine Prado<br>
Registergericht und -nummer: Hamburg, HRB 86891<br>
Sitz der Gesellschaft: Hamburg<br>
</blockquote></div>