[PATCH] D136716: [JumpThreading] Put a limit on the PHI nodes when duplicating a BB.

Usman Nadeem via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 31 11:40:38 PDT 2022


mnadeem added a comment.

In D136716#3896687 <https://reviews.llvm.org/D136716#3896687>, @efriedma wrote:

> This approach seems fine.  Is there any particular reason behind the threshold 76?

I didn't want to miss any threading opportunities if the compile time was reasonable, 76 seemed high enough (after some experimentation) to cover a lot of cases in SPEC2017/cam4 while maintaining a reasonable compile time.


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

https://reviews.llvm.org/D136716



More information about the llvm-commits mailing list