[PATCH] D84940: [JumpThreading] Conditionally freeze its condition when unfolding select

Juneyoung Lee via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 30 06:26:50 PDT 2020


aqjune created this revision.
aqjune added reviewers: efriedma, jdoerfert, nikic, lebedev.ri, spatel.
Herald added subscribers: llvm-commits, dexonsmith, hiraditya.
Herald added a project: LLVM.
aqjune requested review of this revision.

This patch is fixes pr45956 (https://bugs.llvm.org/show_bug.cgi?id=45956 ).
To minimize its impact to the quality of generated code, I suggest enabling
this only for LTO as a start (it has two JumpThreading passes registered).
This patch contains a flag that makes JumpThreading enable it. A patch that makes this run
in LTO passes will be uploaded soon.

To minimize performance degradation, a few updates to JumpThreading were
needed. I'll upload them and add as child patches. This does not resolve all
regressions yet, however.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D84940

Files:
  llvm/include/llvm/Transforms/Scalar.h
  llvm/include/llvm/Transforms/Scalar/JumpThreading.h
  llvm/lib/Transforms/Scalar/JumpThreading.cpp
  llvm/test/Transforms/JumpThreading/select-unfold-freeze.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D84940.281909.patch
Type: text/x-patch
Size: 15265 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200730/3d5d48cb/attachment.bin>


More information about the llvm-commits mailing list