[PATCH] D124252: [SimpleLoopUnswitch] Enable freezing of conditions by default.

[Florian Hahn via Phabricator via llvm-commits]

fhahn created this revision.
fhahn added reviewers: asbirlea, nikic, aeubanks, aqjune, efriedma.
Herald added subscribers: hiraditya, kristof.beyls.
Herald added a project: All.
fhahn requested review of this revision.
Herald added a project: LLVM.

This fixes a series of mis-compiles by SimpleLoopUnswitch.

My measurements showed no performance regression with -O3 on AArch64
in SPEC2006, SPEC2017 and a set of internal benchmarks.

Fixes #50387, #50430

Depends on D124251 <https://reviews.llvm.org/D124251>.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D124252

Files:
  llvm/lib/Transforms/Scalar/SimpleLoopUnswitch.cpp
  llvm/test/Transforms/SimpleLoopUnswitch/ARM/nontrivial-unswitch-cost.ll
  llvm/test/Transforms/SimpleLoopUnswitch/guards.ll
  llvm/test/Transforms/SimpleLoopUnswitch/implicit-null-checks.ll
  llvm/test/Transforms/SimpleLoopUnswitch/nontrivial-unswitch-cost.ll
  llvm/test/Transforms/SimpleLoopUnswitch/nontrivial-unswitch.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D124252.424445.patch
Type: text/x-patch
Size: 17785 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20220422/e3bcfa55/attachment.bin>