diff options
author | Sanjay Patel <spatel@rotateright.com> | 2022-02-13 08:21:57 -0500 |
---|---|---|
committer | Sanjay Patel <spatel@rotateright.com> | 2022-02-13 08:32:21 -0500 |
commit | c486b82cfbe59929a80e5f29bab82112555c8bf4 (patch) | |
tree | a0f95c66f2bc715f863c94726313d492e8a7c424 /llvm/lib/Object/MachOObjectFile.cpp | |
parent | 83ccce6cedbd32f13f73e9805ccbbd1e97dadbfa (diff) | |
download | llvm-c486b82cfbe59929a80e5f29bab82112555c8bf4.zip llvm-c486b82cfbe59929a80e5f29bab82112555c8bf4.tar.gz llvm-c486b82cfbe59929a80e5f29bab82112555c8bf4.tar.bz2 |
[x86] try harder to scalarize a vector load with extracted integer op uses
This is a retry of b4b97ec813a0 - that was reverted because it
could cause miscompiles by illegally reordering memory operations.
A new test based on #53695 is added here to verify we do not have
that same problem.
extract_vec_elt (load X), C --> scalar load (X+C)
As noted in the comment, DAGCombiner has this fold -- and the code in this
patch is adapted from DAGCombiner::scalarizeExtractedVectorLoad() -- but
x86 should benefit even if the loaded vector has other uses as long as we
apply some other x86-specific conditions. The motivating example from #50310
is shown in vec_int_to_fp.ll.
Fixes #50310
Fixes #53695
Differential Revision: https://reviews.llvm.org/D118376
Diffstat (limited to 'llvm/lib/Object/MachOObjectFile.cpp')
0 files changed, 0 insertions, 0 deletions