[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