[PATCH] D153927: Resubmit: [Analysis] Refactor MBB hotness/coldness into templated PSI functions

Han Shen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 28 22:13:35 PDT 2023


shenhan added a comment.

In D153927#4458106 <https://reviews.llvm.org/D153927#4458106>, @MaskRay wrote:

> Thanks. This change should support Bazel builds.
>
>> The origin CL D152758 <https://reviews.llvm.org/D152758> caused gcc-12 build failures, which is caused by a gcc bug, and won't be fixed in version 12.
>
> "CL" means https://opensource.google/documentation/reference/glossary#changelist , but our term is Differential, or just "patch" :)

Done.

> In fact, I'd suggest that you remove
>
>> Below is origin CL description.
>>
>> [NFC] Refactor MBB hotness/coldness into templated PSI functions.
>
> Just start with the original description. You can add information in last paragraph about what's changed from the reverted (two?) commits.

Done

>> In D152399 <https://reviews.llvm.org/D152399>, we calculate BPI->BFI in MachineFunctionSplit pass just to use PSI->isFunctionHotInCallGraph, which is expensive. Instead, we can implement this directly with MBFI.
>
> Since this patch will land before D152399 <https://reviews.llvm.org/D152399> , describing the situation with D152399 <https://reviews.llvm.org/D152399> as the subject may be confusing. I suggest that you describe the current code structure, pretending that D152399 <https://reviews.llvm.org/D152399> does not exist.
> Then, if you think appropriate, describe that this refactoring will make D152399 <https://reviews.llvm.org/D152399> more focused.

Revised the description.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D153927



More information about the llvm-commits mailing list