[PATCH] D22779: Clone block with icmp+branch if it likely results in further jump threading

Haicheng Wu via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 6 20:58:16 PDT 2016


haicheng added a comment.

Hi Easwaran,

My understanding is that your change requires duplicating 2 basic blocks (BB and Succ) to thread over a block.  The normal cases in JumpThreading only need to duplicate 1 block.  Would you mind sharing some performance data if possible?

Other than this, I don't have more comments.  Maybe @mcrosier or @reames has more things to comment.

Haicheng


https://reviews.llvm.org/D22779





More information about the llvm-commits mailing list