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

[Nikita Popov via Phabricator via llvm-commits]

nikic requested changes to this revision.
nikic added a reviewer: reames.
nikic added a comment.
This revision now requires changes to proceed.

Does anything fail if you simply remove this code? We should already be handling inaccessiblememonly generically in AA, there's no need to special-case it here again.

Something I want to note is that we currently do not set `inaccessiblememonly` for `new` in clang (we only set `noalias`), so maybe that should be changed first?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D117180