[PATCH] D31676: [SelectionDAG] [ARM CodeGen] Fix chain information of LowerMUL

Huihui Zhang via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 4 12:15:09 PDT 2017


huihuiz created this revision.
Herald added subscribers: rengolin, aemerson.

In LowerMUL, the chain information is not preserved for the new created Load SDNode.

For example, if a Store alias with one of the operand of Mul. The Load for that operand need to be scheduled before the Store. The dependence is recorded in the chain of Store, in TokenFactor. However, when lowering MUL, the SDNodes for the new Loads for VMULL are not updated in the TokenFactor for the Store. Thus the chain is not preserved for the lowered VMULL.


Repository:
  rL LLVM

https://reviews.llvm.org/D31676

Files:
  lib/Target/ARM/ARMISelLowering.cpp
  test/CodeGen/ARM/lowerMUL-newload.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D31676.94100.patch
Type: text/x-patch
Size: 6159 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170404/c1bb18f3/attachment.bin>


More information about the llvm-commits mailing list