[PATCH] D84941: [JumpThreading] Let ProcessImpliedCondition look into freeze instructions
Nikita Popov via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue May 17 04:44:59 PDT 2022
nikic accepted this revision.
nikic added a comment.
This revision is now accepted and ready to land.
Herald added a project: All.
LGTM
================
Comment at: llvm/lib/Transforms/Scalar/JumpThreading.cpp:1230
+ // V implies Cond, Cond is either true, undef, or poison. In this case,
+ // freeze(Cond) is either true or nondeterministic value.
+ // If freeze(Cond) has only one use, we can freely fold freeze(Cond) to true
----------------
nit: "or a nondeterministic value"
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D84941/new/
https://reviews.llvm.org/D84941
More information about the llvm-commits
mailing list