Page MenuHomePhabricator

Please use GitHub pull requests for new patches. Phabricator shutdown timeline

Differential D39547

[AVX512] Adding support for load truncate store of I1
ClosedPublic
Actions

Authored by m_zuckerman on Nov 2 2017, 6:24 AM.

Details

Reviewers
guyblank
zvi
igorb
ayman
gadi.haber
eladcohen
Commits
rG19fd217eaa74: [AVX512] Adding support for load truncate store of I1
rL320691: [AVX512] Adding support for load truncate store of I1
Summary

store operation on a truncated memory (load) of vXi1 is poorly supported by LLVM and most of the time end with an assertion.
This patch fixes this issue.

Diff Detail

Repository
rL LLVM

Event Timeline

m_zuckerman created this revision.Nov 2 2017, 6:24 AM
m_zuckerman updated this revision to Diff 121299.Nov 2 2017, 7:26 AM
m_zuckerman updated this revision to Diff 121326.Nov 2 2017, 8:56 AM
m_zuckerman updated this revision to Diff 121380.Nov 2 2017, 2:26 PM
igorb added inline comments.Nov 8 2017, 5:52 AM
lib/Target/X86/X86ISelLowering.cpp
1228 ↗(On Diff #121380)

could you please explain why you set custom in case DQI & AVX512 & VL & BWI and not for AVX512

test/CodeGen/X86/avx512-load-trunc-store-i1.ll
4 ↗(On Diff #121380)

Is it possible to create tests that test SELECT only

m_zuckerman updated this revision to Diff 126687.Dec 13 2017, 1:35 AM
m_zuckerman updated this revision to Diff 126711.Dec 13 2017, 3:19 AM
igorb accepted this revision.Dec 13 2017, 4:36 AM

LGTM

This revision is now accepted and ready to land.Dec 13 2017, 4:36 AM
Closed by commit rL320691: [AVX512] Adding support for load truncate store of I1 (authored by mzuckerm). · Explain WhyDec 14 2017, 3:56 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
llvm/
trunk/
lib/
Target/
X86/
3 lines
test/
CodeGen/
X86/
151 lines

Diff 126928

View Options

llvm/trunk/lib/Target/X86/X86ISelLowering.cpp

Loading...
View Options

llvm/trunk/test/CodeGen/X86/avx512-load-trunc-store-i1.ll

Loading...