[PATCH] D117180: [BasicAliasAnalysis] Switch from isMallocOrCallocLikeFn to onlyAccessesInaccessibleMemory

[Nikita Popov via Phabricator via llvm-commits]

nikic added a comment.

It looks like this had some very significant codegen impact, including a 13% code size regression on 7zip: https://llvm-compile-time-tracker.com/compare.php?from=cd97aaee5fef87215eebbd17a31099fe4437ce1f&to=1f2cfc4fdc1eefb2c5f562c77a5fe7e916bbf670&stat=size-text

We should probably revert this until that is investigated. I suspect that it is due to the mentioned issue with `new`, but I didn't expect impact to be this large.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D117180