Re: [Open64-devel] review request -- CG

2012-04-02 Thread Sun Chan
looks fine with ebo fix. for the other fix, shouldn't that be inside x8664? Or at least inside ifdef? Sun On Tue, Apr 3, 2012 at 8:34 AM, Gilmore, Doug wrote: > This patch fixes initializations of several MEM_POOLs in > ebo_special.cxx, which eliminates many memory pool initialization > trace war

[Open64-devel] review request -- CG

2012-04-02 Thread Gilmore, Doug
This patch fixes initializations of several MEM_POOLs in ebo_special.cxx, which eliminates many memory pool initialization trace warnings when the compiler is built in debug mode. Could a gatekeeper review this change when they have a chance? Thanks, Doug ebo_special_pool.patch Description: eb

[Open64-devel] review request -- CG

2012-04-02 Thread Gilmore, Doug
The attached patch fixes a bug in DSP_SCH::Compute_Insn_Size(). Some operations have an opcode variant which assume that rax/eax is an operand/result. Thus when this register is being targeted no Mod R/M byte is needed, thus the instruction is 1 byte shorter. Could a gatekeeper review this chang

[Open64-devel] r3898 - trunk/osprey/be/cg/x8664

2012-04-02 Thread svn
Author: dgilmore Date: 2012-04-02 18:49:39 -0400 (Mon, 02 Apr 2012) New Revision: 3898 Modified: trunk/osprey/be/cg/x8664/ebo_special.cxx Log: This is a fix for a bug found by adding new fields to the OP structure which caused a segmentation fault during the code generation phase when building