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

Dávid Bolvanský via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Sep 7 10:04:20 PDT 2019


xbolva00 added a comment.

I dont have such setup, but some folks could run SPEC for you. Maybe send a RFC about this patch to llvm-dev? :)

I think we really need proper benchmark data before any “LGTM”.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D67318





More information about the llvm-commits mailing list