[PATCH] D100780: [Passes] Add extra LoopSimplifyCFG run after IndVarSimplify.

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 10 09:46:18 PDT 2022


fhahn updated this revision to Diff 428412.
fhahn added a comment.
Herald added subscribers: ormris, wenlei, steven_wu.

Fix failing tests.

This patch increases compile-time overall a bit (details in description). The impact could likely be even small if we only run LoopSimplifyCFG only on loops that indvars changed. This could be done by either exposing a simplifyLoopCFG helper function and use it in indvars or adding a way to mark loops for which LoopSimplifyCFG should run (similar in spirit to D115052 <https://reviews.llvm.org/D115052>).

I'm curious what people think :)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D100780

Files:
  llvm/lib/Passes/PassBuilderPipelines.cpp
  llvm/test/Other/new-pm-defaults.ll
  llvm/test/Other/new-pm-lto-defaults.ll
  llvm/test/Other/new-pm-thinlto-defaults.ll
  llvm/test/Other/new-pm-thinlto-postlink-pgo-defaults.ll
  llvm/test/Other/new-pm-thinlto-postlink-samplepgo-defaults.ll
  llvm/test/Other/new-pm-thinlto-prelink-pgo-defaults.ll
  llvm/test/Other/new-pm-thinlto-prelink-samplepgo-defaults.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D100780.428412.patch
Type: text/x-patch
Size: 5157 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20220510/97df9310/attachment.bin>


More information about the llvm-commits mailing list