This patch looks good to me. Please go ahead. 2012/3/22 Gang Yu <yugang...@gmail.com>: > On Wed, Mar 21, 2012 at 9:27 PM, Jian-Xin Lai <laij...@gmail.com> wrote: >> >> It looks like this branch is redundent? >> + else if (OP_iadd(lda_op)) >> + asm_opnd = OP_opnd( lda_op, 1 ); >> It's fully covered by the else part. > > Yes, thanks. The intended word logic is: under -m32 -fPIC, when we want to > extract the symbol TN from the expanded code, if there are sequences of > > Tmp_TN :- ld32 %ebx (sym:base) > > TN :- lea32 Tmp_TN (base_ofst) > > we build symbol TN of (sym:base + base_ofst) , set the RELOC to > RELOC_IA32_GOTOFF > > if it is a constant, we extrance directly from ldc32, otherwise, we can > always get the symbol TN from the kid1 of lda_op. > > > Regards > Gang >
-- Regards, Lai Jian-Xin ------------------------------------------------------------------------------ This SF email is sponsosred by: Try Windows Azure free for 90 days Click Here http://p.sf.net/sfu/sfd2d-msazure _______________________________________________ Open64-devel mailing list Open64-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/open64-devel
