diff options
author | Ahmed Bougacha <ahmed.bougacha@gmail.com> | 2017-02-03 19:11:19 +0000 |
---|---|---|
committer | Ahmed Bougacha <ahmed.bougacha@gmail.com> | 2017-02-03 19:11:19 +0000 |
commit | 9677cc6fb74918b9f9fffecb226c3747676de868 (patch) | |
tree | 9a9bba7a205977a60649472cf3d39f371fd11c61 /llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp | |
parent | 723999d4aa182bd41c8eb69a5854580a8da5be61 (diff) | |
download | bcm5719-llvm-9677cc6fb74918b9f9fffecb226c3747676de868.tar.gz bcm5719-llvm-9677cc6fb74918b9f9fffecb226c3747676de868.zip |
[TLI] Robustize SDAG LibFunc proto checking by merging it into TLI.
This re-applies commit r292189, reverted in r292191.
SelectionDAGBuilder recognizes libfuncs using some homegrown
parameter type-checking.
Use TLI instead, removing another heap of redundant code.
This isn't strictly NFC, as the SDAG code was too lax.
Concretely, this means changes are required to a few tests:
- calling a non-variadic function via a variadic prototype isn't OK;
it just happens to work on x86_64 (but not on, e.g., aarch64).
- mempcpy has a size_t parameter; the SDAG code accepts any integer
type, which meant using i32 on x86_64 worked.
- a handful of SystemZ tests check the SDAG support for lax prototype
checking: Ulrich agrees on removing them.
I don't think it's worth supporting any of these (IMO) invalid
testcases. Instead, fix them to be more meaningful.
llvm-svn: 294028
Diffstat (limited to 'llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp')
0 files changed, 0 insertions, 0 deletions