[PATCH] D84473: Dump Accumulator

[Kazu Hirata via Phabricator via llvm-commits]

kazu marked an inline comment as done.
kazu added inline comments.


================
Comment at: llvm/include/llvm/Analysis/DumpAccumulator.h:43
+//
+// Note that ThinLTO is not supported yet.
+//
----------------
mtrofin wrote:
> I'd clarify that transferring the data from pre-link thinlto to post-link isn't supported, but one could use this in post-link, correct?
We don't add DumpAccumulator to the post-link ThinLTO pipeline yet.

Note that we add DumpAccumulator in buildPerModuleDefaultPipeline, but post-link ThinLTO doesn't call the function.



Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D84473