[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