[PATCH] D155571: [MemCpyOpt] add terminator user test for D153453(NFC)
    Kohei Asano via Phabricator via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Wed Jul 19 02:08:36 PDT 2023
    
    
  
khei4 added a comment.
@nikic Thank you for the review!
================
Comment at: llvm/test/Transforms/MemCpyOpt/stack-move.ll:417
+
+  %2 = call i32 @use_nocapture(ptr nocapture %dest)
+  call void @llvm.lifetime.end.p0(i64 12, ptr %src)
----------------
nikic wrote:
> Is this call needed to reproduce the crash?
No, just some copy source residue noise! Thanks! 
================
Comment at: llvm/test/Transforms/MemCpyOpt/stack-move.ll:423
+  %lp = landingpad i32 cleanup
+  br label %ehcleanup
+ehcleanup:
----------------
nikic wrote:
> I don't think this extra br is needed, you can directly resume.
Right!
CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D155571/new/
https://reviews.llvm.org/D155571
    
    
More information about the llvm-commits
mailing list