[llvm] [LICM] Use DomTreeUpdater version of SplitBlockPredecessors, nfc (PR #107190)

Yingwei Zheng via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 30 09:00:24 PDT 2024


dtcxzyw wrote:

I noticed that this patch caused a significant compilation time regression (~7%) when building php.
Reproducer: https://github.com/dtcxzyw/llvm-opt-benchmark/blob/main/bench/php/original/parse_date.ll
with commit 6f3c15163f9e: 
```
perf stat -e instructions:u bin/opt -O3 ../../llvm-opt-benchmark/bench/php/original/parse_date.ll --disable-output

Performance counter stats for 'bin/opt -O3 ../../llvm-opt-benchmark/bench/php/original/parse_date.ll --disable-output':

    16,235,016,831      instructions:u                                                        

       2.118263921 seconds time elapsed

       2.101738000 seconds user
       0.015998000 seconds sys
```
with this patch (commit 0bc98349c841):
```
perf stat -e instructions:u bin/opt -O3 ../../llvm-opt-benchmark/bench/php/original/parse_date.ll --disable-output

 Performance counter stats for 'bin/opt -O3 ../../llvm-opt-benchmark/bench/php/original/parse_date.ll --disable-output':

    44,855,633,670      instructions:u                                                        

       5.125998136 seconds time elapsed

       5.106567000 seconds user
       0.019002000 seconds sys
```

See https://github.com/dtcxzyw/llvm-opt-benchmark/issues/1399 for more details.
Can someone have a look?


https://github.com/llvm/llvm-project/pull/107190


More information about the llvm-commits mailing list