[Open64-devel] Review request for fix bug955[CG], a regression caused by r3882

2012-03-16 Thread Gang Yu
Hi, could a gatekeeper please help review the fix for bug955? regression caused by r3882. r3882 causes some regressions on compilation for m32 fpic related applications. For example, open64 triple build failed. a cut down bug-case below: opencc -m32 -fpic -S bug955

Re: [Open64-devel] Review request for fix the O0-DCE bug(bug798)[CG]

2012-03-16 Thread Gang Yu
Thanks for the comment. On Wed, Mar 14, 2012 at 3:32 PM, Jian-Xin Lai wrote: > Some comments: > 1. Link wopt.so into cg.so is bad. It breaks the modularization. Also, > the start time is increased at O0/O1 because wopt.so is loaded > unconditionally. > The essential idea of this patch is to let O

Re: [Open64-devel] Review request for fix the O0-DCE bug(bug798)[CG]

2012-03-16 Thread Gang Yu
On Thu, Mar 15, 2012 at 12:57 PM, Shin-Ming Liu wrote: > I would like to add one more point: > We disable simplifier at -O0 to simplify the debugging at -O0. Any > simplification at -O0 should be justified carefully. > Thanks for the good suggestion, we will make more careful test and approve

Re: [Open64-devel] Review request for fix the O0-DCE bug(bug798)[CG]

2012-03-16 Thread Sun Chan
we are saying, having cg.so depend on wopt.so is NOT the preferred way and we deliberately desing the compiler NOT this way. Your claim that things are moving toward statically linked library. That is not so. You are simply claiming "things are moving backward in time". We deliberately moved to dy

Re: [Open64-devel] Review request for fix the O0-DCE bug(bug798)[CG]

2012-03-16 Thread Shin-Ming Liu
Hi Gang, Your patch proposal is valuable in uncovering some open64 architectural decision made in the early design phase. With that in mind, I try to share some of my point of view. Others are welcome to chip in. There is a goal is to reduce the dependency between cg.so and the rest of the back