Re: [Open64-devel] bug912 fix review request resent

2011-11-29 Thread Sun Chan
I thought whether CVT is needed is decided with the table (I forgot exactly where that is now). This is a machine dependent operation (zero or sign extended when loaded from memory). E.g. IA is zero extended, Mips is sign extended. Or am I in the wrong context? Sun On Tue, Nov 29, 2011 at 4:29 PM,

[Open64-devel] bug912 fix review request resent

2011-11-29 Thread Gang Yu
Hi, list and Sun Let me re-organize the review request for bug912 fix. The bug case: struct go7007 { int sensor_framerate; int fps_scale; }; int vti_bitlen(struct go7007 *go) { unsigned int i, max_time_incr = go->sensor_framerate / go->fps_scale; for (i = 31; (max_time_incr & ((1 << i) -