OSDN Git Service

[ARM][ParallelDSP] Relax alias checks
authorSam Parker <sam.parker@arm.com>
Mon, 13 May 2019 09:23:32 +0000 (09:23 +0000)
committerSam Parker <sam.parker@arm.com>
Mon, 13 May 2019 09:23:32 +0000 (09:23 +0000)
commit17265b988b16d58f9bf576c80a7c5eaa629ea675
tree09fa10463dab637df13f431b4aae8249f8aab212
parent013814dc770c8be05e860d75cdebdfba5a96d4c4
[ARM][ParallelDSP] Relax alias checks

When deciding the safety of generating smlad, we checked for any
writes within the block that may alias with any of the loads that
need to be widened. This is overly conservative because it only
matters when there's a potential aliasing write to a location
accessed by a pair of loads.

Now we check for aliasing writes only once, during setup. If two
loads are found to have an aliasing write between them, we don't add
these loads to LoadPairs. This means that later during the transform,
we can safely widened a pair without worrying about aliasing.

However, to maintain correctness, we also need to change the way that
wide loads are inserted because the order is now important.

The MatchSMLAD method has also been changed, absorbing
MatchReductions and AddMACCandidate to hopefully improve readability.

Differential Revision: https://reviews.llvm.org/D6102

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@360567 91177308-0d34-0410-b5e6-96231b3b80d8
13 files changed:
lib/Target/ARM/ARMParallelDSP.cpp
test/CodeGen/ARM/ParallelDSP/aliasing.ll [new file with mode: 0644]
test/CodeGen/ARM/ParallelDSP/smlad0.ll
test/CodeGen/ARM/ParallelDSP/smlad1.ll
test/CodeGen/ARM/ParallelDSP/smlad11.ll
test/CodeGen/ARM/ParallelDSP/smlad6.ll [deleted file]
test/CodeGen/ARM/ParallelDSP/smlad7.ll [deleted file]
test/CodeGen/ARM/ParallelDSP/smladx-1.ll
test/CodeGen/ARM/ParallelDSP/smlald0.ll
test/CodeGen/ARM/ParallelDSP/smlald1.ll
test/CodeGen/ARM/ParallelDSP/smlald2.ll
test/CodeGen/ARM/ParallelDSP/smlaldx-1.ll
test/CodeGen/ARM/ParallelDSP/smlaldx-2.ll