[PATCH] D112042: [llvm-profgen] Skip duplication factor outside of body sample computation

Wenlei He via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 18 20:29:14 PDT 2021


wenlei added subscribers: rajeshwarv, davidxl.
wenlei added a comment.

@rajeshwarv, checking on google's tool it looks the same issue exists there too - the total sample count could be bloated due to duplication factor. Wondering if the code linked below reflects the latest version used internally, or if there's any special reason for such computation? cc @davidxl

https://github.com/google/autofdo/blob/master/symbol_map.cc#L508

https://github.com/google/autofdo/blob/master/symbol_map.cc#L541-L544


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D112042



More information about the llvm-commits mailing list