SelectionDAG: Make sure stores are always added to the LegalizedNodes list
authorTom Stellard <thomas.stellard@amd.com>
Wed, 21 Aug 2013 22:42:58 +0000 (22:42 +0000)
committerTom Stellard <thomas.stellard@amd.com>
Wed, 21 Aug 2013 22:42:58 +0000 (22:42 +0000)
commitd00968a7a52b23a0d9fc5c1c4b07aceb2cd15006
tree3d3e46179969deba11c303a07012a14a6dbefa8b
parent4db169d31e34df89d10660960070de073f0f28af
SelectionDAG: Make sure stores are always added to the LegalizedNodes list

When truncated vector stores were being custom lowered in
VectorLegalizer::LegalizeOp(), the old (illegal) and new (legal) node pair
was not being added to LegalizedNodes list.  Instead of the legalized
result being passed to VectorLegalizer::TranslateLegalizeResult(),
the result was being passed back into VectorLegalizer::LegalizeOp(),
which ended up adding a (new, new) pair to the list instead.

This was causing an assertion failure when a custom lowered truncated
vector store was the last instruction a basic block and the VectorLegalizer
was unable to find it in the LegalizedNodes list when updating the
DAG root.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@188953 91177308-0d34-0410-b5e6-96231b3b80d8
lib/CodeGen/SelectionDAG/LegalizeVectorOps.cpp
test/CodeGen/R600/trunc-vector-store-assertion-failure.ll [new file with mode: 0644]