[llvm] [NewGVN] Relax conditions when checking safety of memory accesses (PR #98609)
via llvm-commits
llvm-commits at lists.llvm.org
Sat Aug 10 03:31:50 PDT 2024
================
@@ -2597,34 +2597,61 @@ bool NewGVN::OpIsSafeForPHIOfOps(Value *V, const BasicBlock *PHIBlock,
Worklist.push_back(V);
while (!Worklist.empty()) {
auto *I = Worklist.pop_back_val();
- if (!isa<Instruction>(I))
+ if (!(isa<Instruction>(I) || isa<MemoryAccess>(I)))
----------------
ManuelJBrito wrote:
I have updated the PR so that the only MemoryAccesses in the worklist are MemoryPhis.
https://github.com/llvm/llvm-project/pull/98609
More information about the llvm-commits
mailing list