[PATCH] [GVN] Perform Scalar PRE on gep indices that feed loads before doing Load PRE

Daniel Berlin dberlin at dberlin.org
Thu Nov 6 14:23:50 PST 2014


Sounds good to me.
I'll review in detail later today
On Thu Nov 06 2014 at 2:03:14 PM Balaram Makam <bmakam at codeaurora.org>
wrote:

> All,
>
> I updated my comment with slowdowns in compilation times and other
> benchmarks that improve with this patch. Slowdowns in compile times are in
> noise range I do not see any performance regressions greater than 3% in
> Spec. Based on this data, I would like to get rid of the fast path and will
> upload a patch removing out the commented-out code if you agree. Thanks for
> reviewing.
>
> http://reviews.llvm.org/D6103
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20141106/b2f36ec7/attachment.html>


More information about the llvm-commits mailing list