[PATCH] Fix PR16938 - rewrite CSE heuristic for same block uses

Eric Christopher echristo at gmail.com
Tue Apr 29 12:38:02 PDT 2014


Ping :)

-eric

On Tue, Apr 22, 2014 at 4:21 PM, Eric Christopher <echristo at gmail.com> wrote:
> Hi Evan,
>
> In taking a look at 16938 it seems that what you want is to check the
> uses of the original register in all of the basic blocks and then make
> sure that the new CSE'd expression is in one of them rather than the
> inverse?
>
> This will fix the PR mentioned above and no regressions anywhere. I
> ran the performance tests and didn't see anything outside of the noise
> as well.
>
> Thoughts?
>
> -eric



More information about the llvm-commits mailing list