SuccessChanges

Summary

  1. [Attributor] Reuse the IPConstantProp tests for the Attributor (details)
  2. [Attributor] Only replace instruction operands (details)
Commit 3da7efedaa77f8fd8d40294c9e79716003517e6d by johannes
[Attributor] Reuse the IPConstantProp tests for the Attributor
The Attributor can, to some degree, do what IPConstantProp does. We can
consequently use the corner cases already collected and tested for in
the IPConstantProp tests to improve Attributor test coverage.
This exposed various bugs fixed in previous Attributor patches.
Not all functionality of IPConstantProp is available in AAValueSimplify
and AAIsDead so some tests show that we cannot perform the expected
constant propagation.
Reviewers: fhahn, efriedma, mssimpso, davide
Subscribers: bollu, llvm-commits
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D69748
The file was addedllvm/test/Transforms/Attributor/IPConstantProp/arg-type-mismatch.ll
The file was addedllvm/test/Transforms/Attributor/IPConstantProp/comdat-ipo.ll
The file was addedllvm/test/Transforms/Attributor/IPConstantProp/solve-after-each-resolving-undefs-for-function.ll
The file was addedllvm/test/Transforms/Attributor/IPConstantProp/openmp_parallel_for.ll
The file was addedllvm/test/Transforms/Attributor/IPConstantProp/2008-06-09-WeakProp.ll
The file was addedllvm/test/Transforms/Attributor/IPConstantProp/recursion.ll
The file was addedllvm/test/Transforms/Attributor/IPConstantProp/dangling-block-address.ll
The file was addedllvm/test/Transforms/Attributor/IPConstantProp/2009-09-24-byval-ptr.ll
The file was addedllvm/test/Transforms/Attributor/IPConstantProp/multiple_callbacks.ll
The file was addedllvm/test/Transforms/Attributor/IPConstantProp/global.ll
The file was addedllvm/test/Transforms/Attributor/IPConstantProp/arg-count-mismatch.ll
The file was addedllvm/test/Transforms/Attributor/IPConstantProp/thread_local_acs.ll
The file was addedllvm/test/Transforms/Attributor/IPConstantProp/PR43857.ll
The file was addedllvm/test/Transforms/Attributor/IPConstantProp/naked-return.ll
The file was addedllvm/test/Transforms/Attributor/IPConstantProp/return-argument.ll
The file was addedllvm/test/Transforms/Attributor/IPConstantProp/PR26044.ll
The file was addedllvm/test/Transforms/Attributor/IPConstantProp/pthreads.ll
The file was addedllvm/test/Transforms/Attributor/IPConstantProp/deadarg.ll
The file was addedllvm/test/Transforms/Attributor/IPConstantProp/fp-bc-icmp-const-fold.ll
The file was addedllvm/test/Transforms/Attributor/IPConstantProp/musttail-call.ll
The file was addedllvm/test/Transforms/Attributor/IPConstantProp/PR16052.ll
The file was addedllvm/test/Transforms/Attributor/IPConstantProp/remove-call-inst.ll
The file was addedllvm/test/Transforms/Attributor/IPConstantProp/return-constants.ll
The file was addedllvm/test/Transforms/Attributor/IPConstantProp/return-constant.ll
Commit 5d34602da452bd34b6b1bc6bc0a92017050bbe4a by johannes
[Attributor] Only replace instruction operands
This was part of D70767. When we replace the value of (call/invoke)
instructions we do not want to disturb the old call graph so we will
only replace instruction uses until we get rid of the old PM.
Accepted as part of D70767.
The file was modifiedllvm/lib/Transforms/IPO/Attributor.cpp (diff)