[PATCH] D67318: [SimplifyCFG] FoldTwoEntryPHINode(): consider *total* speculation cost, not per-BB cost

Roman Lebedev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 17 01:08:31 PDT 2019


lebedev.ri added a comment.

In D67318#1671803 <https://reviews.llvm.org/D67318#1671803>, @xbolva00 wrote:

> Just curious, did you try threshold 5 or 6 for your rawspeed benchmark?


No, i have not tried any other thresholds here.
I'm personally not aware of any patterns that would benefit from those thresholds, although i did not look.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D67318





More information about the llvm-commits mailing list