diff options
author | Jessica Paquette <jpaquette@apple.com> | 2021-06-29 17:01:28 -0700 |
---|---|---|
committer | Jessica Paquette <jpaquette@apple.com> | 2021-07-12 22:18:50 -0700 |
commit | 47d0780f454d3f7c42bdba13c0682bf2a0095bca (patch) | |
tree | 5c769c46478c06b80b47846818381d6d24d3f68c /llvm/lib/Transforms/Utils/InlineFunction.cpp | |
parent | b8424b42a58ee4cf8ca423de39993ed97f98f0bc (diff) | |
download | llvm-47d0780f454d3f7c42bdba13c0682bf2a0095bca.zip llvm-47d0780f454d3f7c42bdba13c0682bf2a0095bca.tar.gz llvm-47d0780f454d3f7c42bdba13c0682bf2a0095bca.tar.bz2 |
[GlobalISel] Handle more types in narrowScalar for eq/ne G_ICMP
Generalize the existing eq/ne case using `extractParts`. The original code only
handled narrowings for types of width 2n->n. This generalization allows for any
type that can be broken down by `extractParts`.
General overview is:
- Loop over each narrow-sized part and do exactly what the 2-register case did.
- Loop over the leftover-sized parts and do the same thing
- Widen the leftover-sized XOR results to the desired narrow size
- OR that all together and then do the comparison against 0 (just like the old
code)
This shows up a lot when building clang for AArch64 using GlobalISel, so it's
worth fixing. For the sake of simplicity, this doesn't handle the non-eq/ne
case yet.
Also remove the code in this case that notifies the observer; we're just going
to delete MI anyway so talking to the observer shouldn't be necessary.
Differential Revision: https://reviews.llvm.org/D105161
Diffstat (limited to 'llvm/lib/Transforms/Utils/InlineFunction.cpp')
0 files changed, 0 insertions, 0 deletions