bcm5719-llvm/llvm/lib/CodeGen/GlobalISel/CombinerHelper.cpp, branch meklort-10.0.1

[GlobalISel] Change representation of shuffle masks in MachineOperand.

2020-01-14T00:55:41+00:00

We're planning to remove the shufflemask operand from ShuffleVectorInst (D72467); fix GlobalISel so it doesn't depend on that Constant. The change to prelegalizercombiner-shuffle-vector.mir happens because the input contains a literal "-1" in the mask (so the parser/verifier weren't really handling it properly). We now treat it as equivalent to "undef" in all contexts. Differential Revision: https://reviews.llvm.org/D72663

[AArch64][GlobalISel] Fold a chain of two G_PTR_ADDs of constant offsets.

2020-01-07T22:12:42+00:00

E.g. %addr1 = G_PTR_ADD %base, G_CONSTANT 20 %addr2 = G_PTR_ADD %addr1, G_CONSTANT 8 --> %addr2 = G_PTR_ADD %base, G_CONSTANT 28 Differential Revision: https://reviews.llvm.org/D72351

[gicombiner] Import tryCombineIndexedLoadStore()

2019-12-18T14:41:38+00:00

Summary: Now that arbitrary data is supported, import tryCombineIndexedLoadStore() Depends on D69147 Reviewers: bogner, volkan Reviewed By: volkan Subscribers: hiraditya, arphaman, Petar.Avramovic, llvm-commits Tags: #llvm Differential Revision: https://reviews.llvm.org/D69151

[GlobalISel] CombinerHelper: Fix a bug in matchCombineCopy

2019-12-02T20:05:09+00:00

Summary: When combining COPY instructions, we were replacing the destination registers with the source register without checking register constraints. This patch adds a simple logic to check if the constraints match before replacing registers. Reviewers: qcolombet, aditya_nandakumar, aemerson, paquette, dsanders, Petar.Avramovic Reviewed By: aditya_nandakumar Subscribers: rovka, hiraditya, llvm-commits Tags: #llvm Differential Revision: https://reviews.llvm.org/D70616

[GISel][CombinerHelper] Use uses() instead of operands() when traversing use operands.

2019-11-15T21:54:33+00:00

NFC

[GISel][CombinerHelper] Add support for scalar type for the result of shuffle vector

2019-11-15T21:54:33+00:00

LLVM IR of 1-element vectors get lower into scalar in GISel. As a result, shuffle vector may also produce a scalar. This patch teaches the shuffle combiner how to deal with scalars when they are in the destination type of a shuffle vector. For now, we just support the easy case where this can be lowered to a plain copy. For other cases, we leave the shuffle vector as is. This type of IR are seen in O0 pipelines. E.g., as produced with SingleSource/UnitTests/Vector/AArch64/aarch64_neon_intrinsics.c. rdar://problem/57198904

[globalisel] Rename G_GEP to G_PTR_ADD

2019-11-05T18:31:17+00:00

Summary: G_GEP is rather poorly named. It's a simple pointer+scalar addition and doesn't support any of the complexities of getelementptr. I therefore propose that we rename it. There's a G_PTR_MASK so let's follow that convention and go with G_PTR_ADD Reviewers: volkan, aditya_nandakumar, bogner, rovka, arsenm Subscribers: sdardis, jvesely, wdng, nhaehnle, hiraditya, jrtc27, atanasyan, arphaman, Petar.Avramovic, llvm-commits Tags: #llvm Differential Revision: https://reviews.llvm.org/D69734

[GISel][CombinerHelper] Combine shuffle_vector scalar to build_vector

2019-10-31T01:20:37+00:00

Teach the combiner helper how to replace shuffle_vector of scalars into build_vector. I am not particularly happy about having to add this combine, but we currently get those from <1 x iN> from the IR. Bonus: This fixes an assert in the shuffle_vector combines since before this patch, we were expecting vector types.

[GISel][CombinerHelper] Add a combine turning shuffle_vector into concat_vectors

2019-10-21T20:39:58+00:00

Teach the CombinerHelper how to turn shuffle_vectors, that concatenate vectors, into concat_vectors and add this combine to the AArch64 pre-legalizer combiner. Differential Revision: https://reviews.llvm.org/D69149 llvm-svn: 375452

Fix unused variable in r375066

2019-10-17T01:21:40+00:00

llvm-svn: 375070