[PATCH] D149597: [FS-AFDO] Do not load non-FS profile in MIR loader.

Wenlei He via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed May 10 14:15:34 PDT 2023


wenlei added a comment.

> However for lines that do have a zero FSdiscriminator, line+FSdiscriminator can accidentally match the base counter while all other lines still load nothing.

Can this also happen when input profile is actually a FS profile? which is also not good? Or is that less of a problem when input profile is FS because we will also load other samples whose FSDiscriminator is non-zero?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D149597



More information about the llvm-commits mailing list