Replace the result usages while legalizing cmpxchg.
authorLogan Chien <tzuhsiang.chien@gmail.com>
Mon, 21 Jul 2014 17:33:44 +0000 (17:33 +0000)
committerLogan Chien <tzuhsiang.chien@gmail.com>
Mon, 21 Jul 2014 17:33:44 +0000 (17:33 +0000)
commit8c4cf40507abed6efc56b14a13ee291f59a10014
tree8ab0bdc4ec6678db15d590b549dd3e89107db6eb
parentd7858afe790d9c649e4e0785e8a6c8f8d2a71fba
Replace the result usages while legalizing cmpxchg.

We should update the usages to all of the results;
otherwise, we might get assertion failure or SEGV during
the type legalization of ATOMIC_CMP_SWAP_WITH_SUCCESS
with two or more illegal types.

For example, in the following sequence, both i8 and i1
might be illegal in some target, e.g. armv5, mipsel, mips64el,

    %0 = cmpxchg i8* %ptr, i8 %desire, i8 %new monotonic monotonic
    %1 = extractvalue { i8, i1 } %0, 1

Since both i8 and i1 should be legalized, the corresponding
ATOMIC_CMP_SWAP_WITH_SUCCESS dag will be checked/replaced/updated
twice.

If we don't update the usage to *ALL* of the results in the
first round, the DAG for extractvalue might be processed earlier.
The GetPromotedInteger() will result in assertion failure,
because its operand (i.e. the success bit of cmpxchg) is not
promoted beforehand.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@213569 91177308-0d34-0410-b5e6-96231b3b80d8
lib/CodeGen/SelectionDAG/LegalizeIntegerTypes.cpp
test/CodeGen/ARM/atomic-cmpxchg.ll [new file with mode: 0644]
test/CodeGen/Mips/atomic.ll