[PATCH] D144953: [LoopVectorize] Only fetch BFI if profile summary available

Nikita Popov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 28 03:13:05 PST 2023


nikic created this revision.
nikic added reviewers: fhahn, aeubanks.
Herald added subscribers: shiva0217, ormris, StephenFan, wenlei, steven_wu, hiraditya.
Herald added a project: All.
nikic requested review of this revision.
Herald added subscribers: llvm-commits, pcwang-thead.
Herald added a project: LLVM.

BlockFrequencyInfo should generally only be fetched in PGO builds where a PSI profile summary is available. However, LoopVectorize was fetching it unconditionally.

This results in a compile-time improvement: http://llvm-compile-time-tracker.com/compare.php?from=f332498f9880d276890562fb861a375a13bfd9d9&to=7b224f4a8ef2355298051f8f326eccda4cb07b41&stat=instructions%3Au


https://reviews.llvm.org/D144953

Files:
  llvm/include/llvm/Transforms/Vectorize/LoopVectorize.h
  llvm/lib/Transforms/Vectorize/LoopVectorize.cpp
  llvm/test/Other/new-pm-defaults.ll
  llvm/test/Other/new-pm-lto-defaults.ll
  llvm/test/Other/new-pm-thinlto-defaults.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D144953.501068.patch
Type: text/x-patch
Size: 6326 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20230228/88493c3a/attachment.bin>


More information about the llvm-commits mailing list