[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