diff options
| author | Craig Topper <craig.topper@intel.com> | 2018-11-26 21:12:39 +0000 |
|---|---|---|
| committer | Craig Topper <craig.topper@intel.com> | 2018-11-26 21:12:39 +0000 |
| commit | b955bf382cf3ad82c4e8849e898bdfba40ab8aab (patch) | |
| tree | 4ec62477df784839bd46dd46201c44e205c54848 /llvm/lib/CodeGen/SafeStack.cpp | |
| parent | 5f312ad450d1c91fd626fe6ac68ec731553437ea (diff) | |
| download | bcm5719-llvm-b955bf382cf3ad82c4e8849e898bdfba40ab8aab.tar.gz bcm5719-llvm-b955bf382cf3ad82c4e8849e898bdfba40ab8aab.zip | |
[LegalizeVectorTypes][X86][ARM][AArch64][PowerPC] Don't use SplitVecOp_TruncateHelper for FP_TO_SINT/UINT.
SplitVecOp_TruncateHelper tries to promote the result type while splitting FP_TO_SINT/UINT. It then concatenates the result and introduces a truncate to the original result type. But it does this without inserting the AssertZExt/AssertSExt that the regular result type promotion would insert. Nor does it turn FP_TO_UINT into FP_TO_SINT the way normal result type promotion for these operations does. This is bad on X86 which doesn't support FP_TO_SINT until AVX512.
This patch disables the use of SplitVecOp_TruncateHelper for these operations and just lets normal promotion handle it. I've tweaked a couple things in X86ISelLowering to avoid a few obvious regressions there. I believe all the changes on X86 are improvements. The other targets look neutral.
Differential Revision: https://reviews.llvm.org/D54906
llvm-svn: 347593
Diffstat (limited to 'llvm/lib/CodeGen/SafeStack.cpp')
0 files changed, 0 insertions, 0 deletions
