[PATCH] D51983: [ARM] bottom-top mul support in ARMParallelDSP

Sam Parker via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 14 01:07:52 PDT 2018


samparker added inline comments.


================
Comment at: lib/Target/ARM/ARMParallelDSP.cpp:662
+
+  if (AreAliased(AA, Reads, Writes, Candidates))
+    return false;
----------------
SjoerdMeijer wrote:
> I think the alias checks are the same as in MatchSMLAD, so it looks like we can share that, and avoid the duplication here. I could definitely live with that being done in a follow up patch.
Will do.


https://reviews.llvm.org/D51983





More information about the llvm-commits mailing list