[PATCH] D121862: [ProfSampleLoader] When disable-sample-loader-inlining is true, merge profiles of inlined instances to outlining versions.

Mingming Liu via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 17 16:49:45 PDT 2022


luna updated this revision to Diff 416361.
luna added a comment.

Update comment about option --disable-sample-loader-inlining, to call out it doesn't skip profile merging (as the patch currently is).

I'll follow up the discussion (regarding whether this is the desired behavior) in the patch.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D121862

Files:
  llvm/lib/Transforms/IPO/SampleProfile.cpp
  llvm/test/Transforms/SampleProfile/inline-mergeprof.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D121862.416361.patch
Type: text/x-patch
Size: 6376 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20220317/9bb76ecf/attachment.bin>


More information about the llvm-commits mailing list