[PATCH] D59788: [MemDepAnalysis] Allow caller to pass in an OrderedBasicBlock.

[Florian Hahn via Phabricator via llvm-commits]

fhahn added a comment.

In D59788#1444919 <https://reviews.llvm.org/D59788#1444919>, @rnk wrote:

> In D59788#1444721 <https://reviews.llvm.org/D59788#1444721>, @fhahn wrote:
>
> > @rnk is there any chance you still have the full reproducer? I did not manage to reproduce the long time spent in DSE with the snippet above and I cannot share the reproducer I have unfortunately :(
>
>
> Try this bitcode I attached in the tracker:
>  https://bugs.llvm.org/attachment.cgi?id=21689
>
> This is the time report I see for it:
>
>   $ opt -time-passes -O2 longbbinit.bc -o NUL
>   ===-------------------------------------------------------------------------===
>                         ... Pass execution timing report ...
>   ===-------------------------------------------------------------------------===
>     Total Execution Time: 16.4531 seconds (16.4770 wall clock)
>  
>      ---User Time---   --System Time--   --User+System--   ---Wall Time---  --- Name ---
>      9.8281 ( 64.8%)   1.2188 ( 95.1%)  11.0469 ( 67.1%)  11.0376 ( 67.0%)  Dead Store Elimination
>      1.7344 ( 11.4%)   0.0000 (  0.0%)   1.7344 ( 10.5%)   1.6974 ( 10.3%)  Global Value Numbering
>      0.6563 (  4.3%)   0.0156 (  1.2%)   0.6719 (  4.1%)   0.6795 (  4.1%)  Function Integration/Inlining
>      0.3750 (  2.5%)   0.0000 (  0.0%)   0.3750 (  2.3%)   0.3846 (  2.3%)  Value Propagation
>      0.3750 (  2.5%)   0.0000 (  0.0%)   0.3750 (  2.3%)   0.3743 (  2.3%)  Value Propagation #2
>      0.2344 (  1.5%)   0.0156 (  1.2%)   0.2500 (  1.5%)   0.2515 (  1.5%)  Bitcode Writer
>
>
> I'm not sure it's the exact issue you're looking at here, though.


Thanks, that looks promising!

Are you happy with the update to this patch?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D59788/new/

https://reviews.llvm.org/D59788