[Mesa-dev] [PATCH 07/12] R600/SI: move *_Helper definitions to SIInstrFormat.td

Tom Stellard tom at stellard.net
Wed Feb 13 08:15:01 PST 2013


On Wed, Feb 13, 2013 at 10:18:23AM +0100, Christian König wrote:
> Am 13.02.2013 01:20, schrieb Tom Stellard:
> > On Tue, Feb 12, 2013 at 06:13:19PM +0100, Christian König wrote:
> >> From: Christian König <christian.koenig at amd.com>
> >>
> > SIInstrFormats.td should contain the instruction encoding definitions
> > and everything else should go in SIInstrInfo.td.  I got this backwards,
> > when I first created these files, so really these helpers and everything
> > but the encoding definitions should be in SIInstrInfo.td
> 
> Already noticed that there is something mixed up here, but wasn't 100% 
> sure to make a patch.
> 
> Well, I would suggest that we apply this one anyway and then just switch 
> SIInstrInfo.td and SIInstrFormat.td.
>

Ok, this sounds good to me.

> Christian.
> 
> >
> > -Tom
> >
> >> Signed-off-by: Christian König <christian.koenig at amd.com>
> >> ---
> >>   lib/Target/R600/SIInstrFormats.td |   66 +++++++++++++++++++++++++++++++++++++
> >>   lib/Target/R600/SIInstrInfo.td    |   66 -------------------------------------
> >>   2 files changed, 66 insertions(+), 66 deletions(-)
> >>
> >> diff --git a/lib/Target/R600/SIInstrFormats.td b/lib/Target/R600/SIInstrFormats.td
> >> index aea3b5a..7040115 100644
> >> --- a/lib/Target/R600/SIInstrFormats.td
> >> +++ b/lib/Target/R600/SIInstrFormats.td
> >> @@ -144,3 +144,69 @@ class SOPC_32 <bits<7> op, string opName, list<dag> pattern>
> >>   class SOPC_64 <bits<7> op, string opName, list<dag> pattern>
> >>     : SOPC <op, (outs SCCReg:$dst), (ins SReg_64:$src0, SReg_64:$src1), opName, pattern>;
> >>   
> >> +class MIMG_Load_Helper <bits<7> op, string asm> : MIMG <
> >> +  op,
> >> +  (outs VReg_128:$vdata),
> >> +  (ins i32imm:$dmask, i1imm:$unorm, i1imm:$glc, i1imm:$da, i1imm:$r128,
> >> +       i1imm:$tfe, i1imm:$lwe, i1imm:$slc, VReg_32:$vaddr,
> >> +       GPR4Align<SReg_256>:$srsrc, GPR4Align<SReg_128>:$ssamp),
> >> +  asm,
> >> +  []> {
> >> +  let mayLoad = 1;
> >> +  let mayStore = 0;
> >> +}
> >> +
> >> +class MTBUF_Store_Helper <bits<3> op, string asm, RegisterClass regClass> : MTBUF <
> >> +  op,
> >> +  (outs),
> >> +  (ins regClass:$vdata, i16imm:$offset, i1imm:$offen, i1imm:$idxen, i1imm:$glc,
> >> +   i1imm:$addr64, i8imm:$dfmt, i8imm:$nfmt, VReg_32:$vaddr,
> >> +   GPR4Align<SReg_128>:$srsrc, i1imm:$slc, i1imm:$tfe, SReg_32:$soffset),
> >> +  asm,
> >> +  []> {
> >> +  let mayStore = 1;
> >> +  let mayLoad = 0;
> >> +}
> >> +
> >> +class MUBUF_Load_Helper <bits<7> op, string asm, RegisterClass regClass> : MUBUF <
> >> +  op,
> >> +  (outs regClass:$dst),
> >> +  (ins i16imm:$offset, i1imm:$offen, i1imm:$idxen, i1imm:$glc, i1imm:$addr64,
> >> +       i1imm:$lds, VReg_32:$vaddr, GPR4Align<SReg_128>:$srsrc, i1imm:$slc,
> >> +       i1imm:$tfe, SReg_32:$soffset),
> >> +  asm,
> >> +  []> {
> >> +  let mayLoad = 1;
> >> +  let mayStore = 0;
> >> +}
> >> +
> >> +class MTBUF_Load_Helper <bits<3> op, string asm, RegisterClass regClass> : MTBUF <
> >> +  op,
> >> +  (outs regClass:$dst),
> >> +  (ins i16imm:$offset, i1imm:$offen, i1imm:$idxen, i1imm:$glc, i1imm:$addr64,
> >> +       i8imm:$dfmt, i8imm:$nfmt, VReg_32:$vaddr, GPR4Align<SReg_128>:$srsrc,
> >> +       i1imm:$slc, i1imm:$tfe, SReg_32:$soffset),
> >> +  asm,
> >> +  []> {
> >> +  let mayLoad = 1;
> >> +  let mayStore = 0;
> >> +}
> >> +
> >> +multiclass SMRD_Helper <bits<5> op, string asm, RegisterClass dstClass> {
> >> +  def _IMM : SMRD <
> >> +             op, 1,
> >> +             (outs dstClass:$dst),
> >> +             (ins GPR2Align<SReg_64>:$sbase, i32imm:$offset),
> >> +             asm,
> >> +             []
> >> +  >;
> >> +
> >> +  def _SGPR : SMRD <
> >> +              op, 0,
> >> +              (outs dstClass:$dst),
> >> +              (ins GPR2Align<SReg_64>:$sbase, SReg_32:$soff),
> >> +              asm,
> >> +              []
> >> +  >;
> >> +}
> >> +
> >> diff --git a/lib/Target/R600/SIInstrInfo.td b/lib/Target/R600/SIInstrInfo.td
> >> index b983e8a..aa156f3 100644
> >> --- a/lib/Target/R600/SIInstrInfo.td
> >> +++ b/lib/Target/R600/SIInstrInfo.td
> >> @@ -484,71 +484,5 @@ class VOPC <bits<8> op, dag ins, string asm, list<dag> pattern> :
> >>   
> >>   } // End Uses = [EXEC]
> >>   
> >> -class MIMG_Load_Helper <bits<7> op, string asm> : MIMG <
> >> -  op,
> >> -  (outs VReg_128:$vdata),
> >> -  (ins i32imm:$dmask, i1imm:$unorm, i1imm:$glc, i1imm:$da, i1imm:$r128,
> >> -       i1imm:$tfe, i1imm:$lwe, i1imm:$slc, VReg_32:$vaddr,
> >> -       GPR4Align<SReg_256>:$srsrc, GPR4Align<SReg_128>:$ssamp),
> >> -  asm,
> >> -  []> {
> >> -  let mayLoad = 1;
> >> -  let mayStore = 0;
> >> -}
> >> -
> >> -class MUBUF_Load_Helper <bits<7> op, string asm, RegisterClass regClass> : MUBUF <
> >> -  op,
> >> -  (outs regClass:$dst),
> >> -  (ins i16imm:$offset, i1imm:$offen, i1imm:$idxen, i1imm:$glc, i1imm:$addr64,
> >> -       i1imm:$lds, VReg_32:$vaddr, GPR4Align<SReg_128>:$srsrc, i1imm:$slc,
> >> -       i1imm:$tfe, SReg_32:$soffset),
> >> -  asm,
> >> -  []> {
> >> -  let mayLoad = 1;
> >> -  let mayStore = 0;
> >> -}
> >> -
> >> -class MTBUF_Load_Helper <bits<3> op, string asm, RegisterClass regClass> : MTBUF <
> >> -  op,
> >> -  (outs regClass:$dst),
> >> -  (ins i16imm:$offset, i1imm:$offen, i1imm:$idxen, i1imm:$glc, i1imm:$addr64,
> >> -       i8imm:$dfmt, i8imm:$nfmt, VReg_32:$vaddr, GPR4Align<SReg_128>:$srsrc,
> >> -       i1imm:$slc, i1imm:$tfe, SReg_32:$soffset),
> >> -  asm,
> >> -  []> {
> >> -  let mayLoad = 1;
> >> -  let mayStore = 0;
> >> -}
> >> -
> >> -class MTBUF_Store_Helper <bits<3> op, string asm, RegisterClass regClass> : MTBUF <
> >> -  op,
> >> -  (outs),
> >> -  (ins regClass:$vdata, i16imm:$offset, i1imm:$offen, i1imm:$idxen, i1imm:$glc,
> >> -   i1imm:$addr64, i8imm:$dfmt, i8imm:$nfmt, VReg_32:$vaddr,
> >> -   GPR4Align<SReg_128>:$srsrc, i1imm:$slc, i1imm:$tfe, SReg_32:$soffset),
> >> -  asm,
> >> -  []> {
> >> -  let mayStore = 1;
> >> -  let mayLoad = 0;
> >> -}
> >> -
> >> -multiclass SMRD_Helper <bits<5> op, string asm, RegisterClass dstClass> {
> >> -  def _IMM : SMRD <
> >> -             op, 1,
> >> -             (outs dstClass:$dst),
> >> -             (ins GPR2Align<SReg_64>:$sbase, i32imm:$offset),
> >> -             asm,
> >> -             []
> >> -  >;
> >> -
> >> -  def _SGPR : SMRD <
> >> -              op, 0,
> >> -              (outs dstClass:$dst),
> >> -              (ins GPR2Align<SReg_64>:$sbase, SReg_32:$soff),
> >> -              asm,
> >> -              []
> >> -  >;
> >> -}
> >> -
> >>   include "SIInstrFormats.td"
> >>   include "SIInstructions.td"
> >> -- 
> >> 1.7.9.5
> >>
> 


More information about the mesa-dev mailing list