diff options
| author | Alex Bradbury <asb@lowrisc.org> | 2017-10-11 13:48:45 +0000 |
|---|---|---|
| committer | Alex Bradbury <asb@lowrisc.org> | 2017-10-11 13:48:45 +0000 |
| commit | 4d275f0dfec335e04b64461556205c965079d6d7 (patch) | |
| tree | 104f29bb20a65d37e147d90866795b2003b021a9 /llvm/lib/Transforms | |
| parent | 8acb2e3ac40982acb49b894972b11f00ed72598c (diff) | |
| download | bcm5719-llvm-4d275f0dfec335e04b64461556205c965079d6d7.tar.gz bcm5719-llvm-4d275f0dfec335e04b64461556205c965079d6d7.zip | |
[TargetLowering] Correctly track NumFixedArgs field of CallLoweringInfo
The NumFixedArgs field of CallLoweringInfo is used by
TargetLowering::LowerCallTo to determine whether a given argument is passed
using the vararg calling convention or not (specifically, to set IsFixed for
each ISD::OutputArg).
Firstly, CallLoweringInfo::setLibCallee and CallLoweringInfo::setCallee both
incorrectly set NumFixedArgs based on the _previous_ args list. Secondly,
TargetLowering::LowerCallTo failed to increment NumFixedArgs when modifying
the argument list so a pointer is passed for the return value.
If your backend uses the IsFixed property or directly accesses NumFixedArgs,
it is _possible_ this change could result in codegen changes (although the
previous behaviour would have been incorrect). No such cases have been
identified during code review for any in-tree architecture.
Differential Revision: https://reviews.llvm.org/D37898
llvm-svn: 315457
Diffstat (limited to 'llvm/lib/Transforms')
0 files changed, 0 insertions, 0 deletions

