On Die, 2013-02-12 at 18:13 +0100, Christian König wrote:
> From: Christian König <christian.koe...@amd.com>
>
> Mark all the operands that can also have an immediate.
>
> Signed-off-by: Christian König <christian.koe...@amd.com>
> ---
> lib/Target/R600/SIInstrFormats.td | 32 +++++-----
> lib/Target/R600/SIInstructions.td | 128
> ++++++++++++++++++-------------------
> lib/Target/R600/SIRegisterInfo.td | 10 ++-
> 3 files changed, 87 insertions(+), 83 deletions(-)
>
> diff --git a/lib/Target/R600/SIInstrFormats.td
> b/lib/Target/R600/SIInstrFormats.td
> index a7a3558..473d3dc 100644
> --- a/lib/Target/R600/SIInstrFormats.td
> +++ b/lib/Target/R600/SIInstrFormats.td
> @@ -22,25 +22,25 @@
>
> //===----------------------------------------------------------------------===//
>
> class VOP3_32 <bits<9> op, string opName, list<dag> pattern>
> - : VOP3 <op, (outs VReg_32:$dst), (ins AllReg_32:$src0, VReg_32:$src1,
> VReg_32:$src2, i32imm:$src3, i32imm:$src4, i32imm:$src5, i32imm:$src6),
> opName, pattern>;
> + : VOP3 <op, (outs VReg_32:$dst), (ins VSrc_32:$src0, VReg_32:$src1,
> VReg_32:$src2, i32imm:$src3, i32imm:$src4, i32imm:$src5, i32imm:$src6),
> opName, pattern>;
Note that I think all source operands can use inline constants, even
several different ones. Also, the same SGPR can be used for several
source operands. The only limitation is that at most one actual SGPR or
literal constant can be used by the VALU. Not sure how to model the
latter more accurately, but I think the former might be relatively
straightforward.
BTW, please rebase this series on top of the output modifier fix I
pushed to the LLVM trunk, it added a couple more uses of SREG_LIT_0,
which I'm not sure how to properly merge with your changes.
P.S. I noticed this warning that isn't there without your series:
.../lib/Target/R600/MCTargetDesc/SIMCCodeEmitter.cpp: In member function
‘virtual void {anonymous}::SIMCCodeEmitter::EncodeInstruction(const
llvm::MCInst&, llvm::raw_ostream&, llvm::SmallVectorImpl<llvm::MCFixup>&)
const’:
.../lib/Target/R600/MCTargetDesc/SIMCCodeEmitter.cpp:174:53: warning:
‘Imm.{anonymous}::IntFloatUnion::F’ may be used uninitialized in this function
[-Wmaybe-uninitialized]
I guess it can't tell that one of the union members will always be
initialized. Maybe convert
else if (Op.isFPImm())
Imm.F = Op.getFPImm();
to something like
else {
assert(Op.isFPImm());
Imm.F = Op.getFPImm();
}
--
Earthling Michel Dänzer | http://www.amd.com
Libre software enthusiast | Debian, X and DRI developer
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
