[PATCH] D125255: [llvm-profgen] Support a threshold to control hiding warning summaries

[Hongtao Yu via Phabricator via llvm-commits]

hoy added a comment.

In D125255#3502157 <https://reviews.llvm.org/D125255#3502157>, @wenlei wrote:

> I'm not sure if a universal threshold for filtering warning is a good way to deal with this. Maybe look at this from a different angle -- for some warnings, any instance is bad, for others, we can tolerate them. Hence, an alternative: for critical ones, we should always print out any non-zero value; for others, print them only under a "verbose" mode?

That's fair. It'd be good to treat warnings of different importance differently.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D125255