Notably helps cleanup after legalization of vector types
Details
Diff Detail
- Repository
- rL LLVM
Event Timeline
lib/CodeGen/SelectionDAG/TargetLowering.cpp | ||
---|---|---|
1359 | rename to SrcDemandedElts? | |
1361–1362 | I think this would read more clearly as: | |
1385–1386 | Similar to above: | |
1398 | if -> it | |
test/CodeGen/X86/vector-shuffle-256-v16.ll | ||
4606–4609 | I haven't looked at this in a while, so not sure what's happening here, but this should just be: |
test/CodeGen/X86/vector-shuffle-256-v16.ll | ||
---|---|---|
4606–4609 | Its only broadcasting the lower i16 of the i32, so no unfortunately - in fact I think the test was added to make sure we don't do this. |
LGTM.
test/CodeGen/X86/vector-shuffle-256-v16.ll | ||
---|---|---|
4606–4609 | Oops, I should've noticed that from the AVX2 code. |
rename to SrcDemandedElts?