[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