[PATCH] D105524: [FuncSpec] Extract the analysis part as an analysis pass

Chuanqi Xu via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 7 01:15:50 PDT 2021


ChuanqiXu added a comment.

In D105524#2861297 <https://reviews.llvm.org/D105524#2861297>, @SjoerdMeijer wrote:

> Ah, I remembered your post to the mailing list about this which provides more context:
>
> https://lists.llvm.org/pipermail/llvm-dev/2021-May/150443.html
>
> Looks like you're raising fundamental ThinLTO questions that need answering first. Perhaps ping that thread, perhaps update it with some more information, so that hopefully someone with ThinLTO knowledge can shed a light on this approach.

Yeah, I posted it before.

Then  I made it in the downstream to see the impact I raised in the mailing list. And the impact looks good from my experiments. So I want to see if it is OK to contribute them to the upstream.

Since I want to make things happen in order, so I updated this patch first.  Now it looks like I'd better to update the remained patch and discuss in the thread.

I would do that. Thanks for suggestion.


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

https://reviews.llvm.org/D105524



More information about the llvm-commits mailing list