[PATCH] D98835: [SampleFDO] Don't mix up the existing indirect call value profile with the new value profile annotated after inlining.

Hongtao Yu via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 18 10:16:07 PDT 2021


hoy added a comment.

Thanks for fixing the assert failure!



================
Comment at: llvm/test/Transforms/SampleProfile/norepeated-icp-3.ll:71
+; CHECK: ![[PROF_ID]] = !{!"VP", i32 0, i64 5860, i64 -7383239051784516332, i64 -1, i64 -7701940972712279918, i64 5860}
+!22 = !{!"VP", i32 0, i64 8000, i64 -7383239051784516332, i64 -1, i64 125292384912345234234, i64 8000}
----------------
I'd like to understand a bit more about the counts here. In reality, is 8000 from prelink and 5860 from postlink? If the same profile is used, why different callees can be seen?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D98835



More information about the llvm-commits mailing list