[PATCH] D61780: [ARM][ParallelDSP] Change the search for smlads

Fri May 10 02:12:24 PDT 2019

samparker created this revision.
samparker added a reviewer: SjoerdMeijer.
Herald added subscribers: zzheng, kristof.beyls, javed.absar.
samparker added a parent revision: D61020: [ARM][ParallelDSP] Relax alias checks.

Two functional changes have been made here:

1. Now search up from any add instruction to find the chains of operations that we may turn into a smlad. This allows the generation of a smlad which doesn't accumulate into a phi.
2. The search function has been corrected to stop it falsely searching up through an invalid path.

The bulk of the changes have been making the Reduction struct a class and making it more C++y with getters and setters.

https://reviews.llvm.org/D61780

Files:
  lib/Target/ARM/ARMParallelDSP.cpp
  test/CodeGen/ARM/ParallelDSP/aliasing.ll
  test/CodeGen/ARM/ParallelDSP/inner-full-unroll.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D61780.198991.patch
Type: text/x-patch
Size: 30743 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190510/f04728a5/attachment.bin>