[PATCH] D32252: [GVN] Add phi-translate for scalarpre as a temporary solution

[Wei Mi via llvm-commits]

On Mon, May 15, 2017 at 10:04 AM, Davide Italiano via Phabricator
<reviews at reviews.llvm.org> wrote:
> davide added a comment.
>
> In https://reviews.llvm.org/D32252#750282, @wmi wrote:
>
>> I change the DenseMap numberingExpression (mapping from value numbering to GVN::Expression) to std::vector. Although there is still some minor increase but better than before.
>>
>> 400.perlbench 0.81%
>>  401.bzip2    -0.11%
>>  403.gcc      -0.42%
>>  429.mcf      1.58%
>>  433.milc     -0.33%
>>  445.gobmk    0.85%
>>  456.hmmer    0.07%
>>  458.sjeng    -0.42%
>>  462.libquantum       -0.75%
>>  464.h264ref  -1.24%
>>  470.lbm      -0.65%
>>  482.sphinx3  0.44%
>>  998.specrand 0.00%
>>  999.specrand -1.74%
>>  444.namd     0.28%
>>  447.dealII   1.56%
>>  450.soplex   0.96%
>>  453.povray   -0.02%
>>  471.omnetpp  0.02%
>>  473.astar    0.18%
>>  483.xalancbmk        0.85%
>
>
> The compile time numbers don't look particularly bad. Thanks.
> Can you try on large testcases/LTO (maybe of clang itself) to confirm that scales in a sane way?

I bootstrapped clang. The compile time (make -j6, user time) "without
vs with" the patch is: 12464.65 vs 12483.85.
I am now doing the LTO bootstrap now.

>
>
> Repository:
>   rL LLVM
>
> https://reviews.llvm.org/D32252
>
>
>