[PATCH] D137553: [MemorySSA] Delete dead MemoryUseOrDef for CallInst when clone loop basicblock

luxufan via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 7 18:53:20 PST 2022


StephenFan added inline comments.


================
Comment at: llvm/test/Transforms/SimpleLoopUnswitch/pr58719.ll:1
+; RUN: opt -passes="require<globals-aa>,cgscc(instcombine),recompute-globalsaa,function(loop-mssa(simple-loop-unswitch<nontrivial>),print<memoryssa>)" -disable-output < %s 2>&1 | FileCheck %s
+define void @f() {
----------------
fhahn wrote:
> is instcombine needed here?
Yes. InstCombine will delete the load instruction in function `f`.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D137553/new/

https://reviews.llvm.org/D137553



More information about the llvm-commits mailing list