[PATCH] D112858: [llvm-profgen] Fill zero count for all function ranges

[Wenlei He via Phabricator via llvm-commits]

wenlei added inline comments.


================
Comment at: llvm/test/tools/llvm-profgen/inline-noprobe.test:7
 ; RUN: FileCheck %s --input-file %t1 --check-prefix=CHECK
+; RUN: llvm-profgen --format=text --unsymbolized-profile=%t --binary=%S/Inputs/inline-noprobe.perfbin --output=%t1 --fill-zero-for-all-funcs
+; RUN: FileCheck %s --input-file %t1 --check-prefix=CHECK
----------------
modimo wrote:
> hoy wrote:
> > Does it work when given an empty raw profile? Can it also work without giving a profile input? That can be a real scenario. 
> No profile here would be great!
I think asking user to provide an (empty) profile is reasonable.. Otherwise it feels a bit stretched for llvm-profgen as a profile generation tool. 


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D112858