OSDN Git Service

R600/SI: Fix bug where immediates were being used in DS addr operands
authorTom Stellard <thomas.stellard@amd.com>
Wed, 15 Oct 2014 21:08:59 +0000 (21:08 +0000)
committerTom Stellard <thomas.stellard@amd.com>
Wed, 15 Oct 2014 21:08:59 +0000 (21:08 +0000)
commitd3fc10a52532a8da060f8d00a9c545e0a6ce156e
treef4695c535b55d3df2190946fd2d1597e39bb3ffe
parent55751ba9e2c0d1fb0ff71fb74f5fb922ea29f2e5
R600/SI: Fix bug where immediates were being used in DS addr operands

The SelectDS1Addr1Offset complex pattern always tries to store constant
lds pointers in the offset operand and store a zero value in the addr operand.
Since the addr operand does not accept immediates, the zero value
needs to first be copied to a register.

This newly created zero value will not go through normal instruction
selection, so we need to manually insert a V_MOV_B32_e32 in the complex
pattern.

This bug was hidden by the fact that if there was another zero value
in the DAG that had not been selected yet, then the CSE done by the DAG
would use the unselected node for the addr operand rather than the one
that was just created.  This would lead to the zero value being selected
and the DAG automatically inserting a V_MOV_B32_e32 instruction.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@219848 91177308-0d34-0410-b5e6-96231b3b80d8
lib/Target/R600/AMDGPUISelDAGToDAG.cpp
test/CodeGen/R600/load.ll