[Beignet] [PATCH] output float immediate value with %f instead of %g

Guo, Yejun yejun.guo at intel.com
Mon Aug 3 22:40:53 PDT 2015


Got it, thanks.

The ASM is outputted for human reading, it is expected that a correct and human readable format is used.

-----Original Message-----
From: Song, Ruiling 
Sent: Tuesday, August 04, 2015 12:41 PM
To: Guo, Yejun; beignet at lists.freedesktop.org
Subject: RE: [Beignet] [PATCH] output float immediate value with %f instead of %g



> -----Original Message-----
> From: Guo, Yejun
> Sent: Tuesday, August 4, 2015 10:50 AM
> To: Song, Ruiling; beignet at lists.freedesktop.org
> Subject: RE: [Beignet] [PATCH] output float immediate value with %f 
> instead of %g
> 
> Yes, it would be better, not sure why the current code generates "F" 
> for the mov instruction, I'll take a look.
The code generator select float type instead of integer type because IVB has more floating point ALU over integer ones.
See LoadImm pattern in gen_insn_selection.cpp But I don't think this is a performance sensitive point.

Thanks!
Ruiling


More information about the Beignet mailing list