[PATCH] D109531: [CSSPGO] Enable pseudo probe instrumentation in O0 mode.

Thu Sep 9 14:20:27 PDT 2021

hoy added a comment.

In D109531#2992702 <https://reviews.llvm.org/D109531#2992702>, @wenlei wrote:

> The change makes sense given instr PGO also happens for O0. But practically, if a file is being built with O0, do we care about its profile given we're not really optimizing it anyways? Functions from O0 modules are not supposed to be inlined into O1 <https://reviews.llvm.org/owners/package/1/>+ modules either.

We probably don't care about performance for O0 build. The change is for consistency, also makes the compiler happy which otherwise will complain about "Pseudo-probe-based profile requires SampleProfileProbePass" for O0 modules that don't have probes.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D109531