[PATCH] D154307: [InstructionSimplify] Avoid simplifying ICmp without parent
Manuel Brito via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Aug 10 05:24:57 PDT 2023
ManuelJBrito added a comment.
Perhaps we could add this case as a check to OpIsSafeForPHIOfOps. While walking the operands we could check if they are used in an assume until we hit a dominator. Not sure this makes complete sense, but we already do something similar to prevent InstSimplify from code walking through non-dominating phis.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D154307/new/
https://reviews.llvm.org/D154307
More information about the llvm-commits
mailing list