[PATCH] D107513: [MemCpyOpt/MemorySSA] Do not run the pass for prohibitively large number of memory accesses.
    Alina Sbirlea via Phabricator via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Mon Aug  9 17:34:06 PDT 2021
    
    
  
asbirlea abandoned this revision.
asbirlea added a comment.
I don't think we need this patch for the specific issue that motivated it. It's possible it will be useful for other passes that insert defs and end up in a similar scenario though.
At this point it's not worth pursuing and I think I prefer to analyze the uses of the inserDef API instead and reopen if a use case makes sense (e.g. there is a capping done in LICM, and that one iterates through the access list).
Repository:
  rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D107513/new/
https://reviews.llvm.org/D107513
    
    
More information about the llvm-commits
mailing list