diff options
| author | Craig Topper <craig.topper@intel.com> | 2018-01-29 07:52:55 +0000 |
|---|---|---|
| committer | Craig Topper <craig.topper@intel.com> | 2018-01-29 07:52:55 +0000 |
| commit | 62b62356fae9a66a7bdbf33d5790d7db39c9af5e (patch) | |
| tree | 60b62197fd34bfe11c9ab719a3c05f4885593362 /llvm/lib/Transforms/IPO/FunctionImport.cpp | |
| parent | 8b797a0fd25d8000b239c76016833ad4ab81339b (diff) | |
| download | bcm5719-llvm-62b62356fae9a66a7bdbf33d5790d7db39c9af5e.tar.gz bcm5719-llvm-62b62356fae9a66a7bdbf33d5790d7db39c9af5e.zip | |
[X86] Make foldLogicOfSetCCs work better for vectors pre legal types/operations
Summary:
There's a check in the code to only check getSetCCResultType after LegalOperations or if the type is MVT::i1. But the i1 check is only allowing scalar types through. I think it should check that the scalar type is MVT::i1 so that it will work for vectors.
The changed test already does this combine with AVX512VL where getSetCCResultType returns vXi1. But with avx512f and no VLX getSetCCResultType returns a type matching the width of the input type.
Reviewers: spatel, RKSimon
Reviewed By: spatel
Subscribers: llvm-commits
Differential Revision: https://reviews.llvm.org/D42619
llvm-svn: 323631
Diffstat (limited to 'llvm/lib/Transforms/IPO/FunctionImport.cpp')
0 files changed, 0 insertions, 0 deletions
