[Attributor][FIX] Remove duplicate, half-broken functionality
authorJohannes Doerfert <johannes@jdoerfert.de>
Sun, 12 Jan 2020 06:17:08 +0000 (00:17 -0600)
committerJohannes Doerfert <johannes@jdoerfert.de>
Tue, 11 Feb 2020 17:02:59 +0000 (11:02 -0600)
commit4c62a358602e88abc275009bd4170b891eb07870
treed046bf90cf6aedf89fef8b5bfaf2bf007defda33
parent77a9e61c9ac89e93809789f904085dbf365829c4
[Attributor][FIX] Remove duplicate, half-broken functionality

The changeXXXAfterManifest functions are better suited to deal with
changes so we should prefer them. These functions also recursively
delete dead instructions which is why we see test changes.
12 files changed:
llvm/include/llvm/Transforms/IPO/Attributor.h
llvm/lib/Transforms/IPO/Attributor.cpp
llvm/test/Transforms/Attributor/ArgumentPromotion/live_called_from_dead.ll
llvm/test/Transforms/Attributor/ArgumentPromotion/musttail.ll
llvm/test/Transforms/Attributor/IPConstantProp/comdat-ipo.ll
llvm/test/Transforms/Attributor/IPConstantProp/remove-call-inst.ll
llvm/test/Transforms/Attributor/IPConstantProp/return-constant.ll
llvm/test/Transforms/Attributor/IPConstantProp/solve-after-each-resolving-undefs-for-function.ll
llvm/test/Transforms/Attributor/liveness.ll
llvm/test/Transforms/Attributor/nonnull.ll
llvm/test/Transforms/Attributor/range.ll
llvm/test/Transforms/Attributor/undefined_behavior.ll