[PATCH] D109860: profi - a flow-based profile inference algorithm: Part I (out of 3)

[Hongtao Yu via Phabricator via llvm-commits]

hoy added a comment.

In D109860#3149536 <https://reviews.llvm.org/D109860#3149536>, @mehdi_amini wrote:

> I see the definition being `cl::opt<bool> SampleProfileUseProfi(` while the declaration is `extern cl::opt<unsigned> SampleProfileUseProfi;`

Good catch! That should be the culprit. The declaration should be cl::opt<bool> as well. Could you kindly test it for us since we don't have a Windows environment? Thanks!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D109860