> 
> In disassemble, a float immediate value is printf with %g, it outputs a wrong
> value with 8 bytes, change it to be %f with the correct 4 bytes. At some 
> cases,
> the real data type is not float, so also print the value in hex format.

As far as I know, %g will select %f or %e automatically, which is good for 
formatting.
The compiler would convert the float to double, then printf will output the 
double (8 bytes) value.
I am not quite sure on this. So I don't think this will bring in any issue. 
What kind of problem do you meet?

I agree with output the hex value at first glance of the patch, but the output 
asm would looks like
a little messy, so I don't think it is a good idea to output the hex value.
    (    1308)  mul(16)         g110<1>:F       g112<8,8,1>:F   1e-30F 
(0xda24260) { align1 WE_normal 1H };

Thanks!
Ruiling
_______________________________________________
Beignet mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/beignet

Reply via email to