[PATCH] D60295: [SelectionDAG] Change the jump table size unit from entry to target

Evandro Menezes via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 25 06:25:35 PDT 2019


evandro added a comment.

Yes, `default`, whether explicit or implicit, counts too,  for the resources used inside the target only care about different addresses.  In you example, a typical branch predictor will see up to 3 target addresses.  Of course, branch predictors are trained by branches that are actually taken, so only target addresses that are executed count.  Which opens the window for future work involving FDO or just the static heuristics.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D60295





More information about the llvm-commits mailing list