[PATCH] D54254: [ARM] Reorganise some functionality in ARMParallelDSP

[Sam Parker via Phabricator via llvm-commits]

samparker created this revision.
samparker added a reviewer: SjoerdMeijer.
Herald added subscribers: chrib, kristof.beyls, javed.absar.

A few code movement things:

- AreSymmetrical is now a method of BinOpChain.
- Created a lambda in CreateParallelMACPairs to reduce loop nesting.
- A Reduction object now gets pasted in a couple of places instead, including CreateParallelMACPairs so it doesn't need to return a value.

I've also added RecordSequentialLoads, which is run before the transformation begins, and caches the interesting loads. This can then be queried later instead of cross checking many load values.


https://reviews.llvm.org/D54254

Files:
  lib/Target/ARM/ARMParallelDSP.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D54254.173138.patch
Type: text/x-patch
Size: 13366 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20181108/a698de27/attachment.bin>