2016-10-26 19:20 GMT+02:00 Samuel Pitoiset <samuel.pitoi...@gmail.com>:
>
>
> On 10/09/2016 11:04 AM, Karol Herbst wrote:
>>
>> v2: renamed commit
>> reordered modifiers
>> add assert(dst == src2)
>>
>> Signed-off-by: Karol Herbst <karolher...@gmail.com>
>> ---
>> .../drivers/nouveau/codegen/nv50_ir_emit_gm107.cpp | 35
>> ++++++++++++++++------
>> 1 file changed, 26 insertions(+), 9 deletions(-)
>>
>> diff --git a/src/gallium/drivers/nouveau/codegen/nv50_ir_emit_gm107.cpp
>> b/src/gallium/drivers/nouveau/codegen/nv50_ir_emit_gm107.cpp
>> index 3fedafd..5729615 100644
>> --- a/src/gallium/drivers/nouveau/codegen/nv50_ir_emit_gm107.cpp
>> +++ b/src/gallium/drivers/nouveau/codegen/nv50_ir_emit_gm107.cpp
>> @@ -1306,7 +1306,7 @@ CodeEmitterGM107::emitFMUL()
>> void
>> CodeEmitterGM107::emitFFMA()
>> {
>> - /*XXX: ffma32i exists, but not using it as third src overlaps dst */
>> + bool isLongIMMD = false;
>> switch(insn->src(2).getFile()) {
>> case FILE_GPR:
>> switch (insn->src(1).getFile()) {
>> @@ -1319,14 +1319,22 @@ CodeEmitterGM107::emitFFMA()
>> emitCBUF(0x22, -1, 0x14, 16, 2, insn->src(1));
>> break;
>> case FILE_IMMEDIATE:
>> - emitInsn(0x32800000);
>> - emitIMMD(0x14, 19, insn->src(1));
>> + if (longIMMD(insn->getSrc(1))) {
>> + assert(insn->getDef(0)->reg.data.id ==
>> insn->getSrc(2)->reg.data.id);
>> + isLongIMMD = true;
>> + emitInsn(0x0c000000);
>> + emitIMMD(0x14, 32, insn->src(1));
>> + } else {
>> + emitInsn(0x32800000);
>> + emitIMMD(0x14, 19, insn->src(1));
>> + }
>> break;
>> default:
>> assert(!"bad src1 file");
>> break;
>> }
>> - emitGPR (0x27, insn->src(2));
>> + if (!isLongIMMD)
>> + emitGPR (0x27, insn->src(2));
>
>
> Because def(0) == src(2), you can eventually drop that check and call
> emitGPR() even for the LIMM form.
emitIMMD does it already for the def though, so it would be kind of
pointless to do it again.
>
>> break;
>> case FILE_MEMORY_CONST:
>> emitInsn(0x51800000);
>> @@ -1337,11 +1345,20 @@ CodeEmitterGM107::emitFFMA()
>> assert(!"bad src2 file");
>> break;
>> }
>> - emitRND (0x33);
>> - emitSAT (0x32);
>> - emitNEG (0x31, insn->src(2));
>> - emitNEG2(0x30, insn->src(0), insn->src(1));
>> - emitCC (0x2f);
>> +
>> + emitRND(0x33);
>> +
>> + if (isLongIMMD) {
>> + emitSAT (0x37);
>> + emitNEG (0x39, insn->src(2));
>> + emitNEG2(0x38, insn->src(0), insn->src(1));
>> + emitCC (0x34);
>
>
> Order:
> 0x39
> 0x38
> 0x37
> 0x34
>
>> + } else {
>> + emitSAT (0x32);
>> + emitNEG (0x31, insn->src(2));
>> + emitNEG2(0x30, insn->src(0), insn->src(1));
>> + emitCC (0x2f);
>> + }
>>
>> emitFMZ(0x35, 2);
>> emitGPR(0x08, insn->src(0));
>>
>
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
