[PATCH] D150375: [FuncSpec] Replace LoopInfo with BlockFrequencyInfo.

Chuanqi Xu via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 16 02:45:44 PDT 2023


ChuanqiXu added a comment.

In D150375#4345226 <https://reviews.llvm.org/D150375#4345226>, @labrinea wrote:

> In D150375#4337042 <https://reviews.llvm.org/D150375#4337042>, @ChuanqiXu wrote:
>
>> Interesting. Have you measured with spec2017?
>
> Hi. I measured (without PGO) but didn't see any changes. The main drive for this change is to enable specialization of literal constants (D150649 <https://reviews.llvm.org/D150649>), which would otherwse trigger too often with LoopInfo.

Do you measured it with fulllto? If yes, I feel the change looks good.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D150375



More information about the llvm-commits mailing list