[DSE] Remove alloc function check in canSkipDef()
authorNikita Popov <npopov@redhat.com>
Tue, 11 Jan 2022 11:34:11 +0000 (12:34 +0100)
committerNikita Popov <npopov@redhat.com>
Mon, 17 Jan 2022 08:23:51 +0000 (09:23 +0100)
commit00b77d917cd8e062b93e080f35cfd5aa15348f31
tree5cd5685530dff12ce36cd456a490895cff0476eb
parent4b22ffe0b96ab962ff454d22b8f2170c7d26b452
[DSE] Remove alloc function check in canSkipDef()

canSkipDef() currently skips inaccessiblememonly calls, but not
if they are allocation functions. This check was added in D103009,
but actually seems to be a leftover from a previous implementation
in D101440. canSkipDef() is not used on the storeIsNoop() path,
where the relevant transform ended up being implemented.

Differential Revision: https://reviews.llvm.org/D117005
llvm/lib/Transforms/Scalar/DeadStoreElimination.cpp