[PATCH] D97623: [SampleFDO] Add a cutoff flag to control how many symbols will be included into profile symbol list.

[David Li via Phabricator via llvm-commits]

davidxl added inline comments.


================
Comment at: llvm/test/Transforms/SampleProfile/profile-sample-accurate.ll:66
+; Check -profile-symbol-list-cut-off=2 will include _Z3toov into profile
+; symbol list and -profile-symbol-list-cut-off=3 will not.
+; PSLCUTOFF2: define i32 @_Z3toov{{.*}}!prof ![[TOO_ID:[0-9]+]]
----------------
tejohnson wrote:
> Shouldn't it be the other way around - i.e. with the larger cutoff it will be included and with the smaller one it isn't?
> 
> Also - confused at how the cutoff correlates to the placement in profile-symbol-list.text. In that file, _Z3toov is symbol number 5. So wouldn't a cutoff of 4 vs 5 affect whether it is included?
I think it is sorted list.

When cutoff is 3, precise symbol list includes too, so too has profile of 0.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D97623