Re: [PATCH, combine] Fix host-specific behavior in simplify_compare_const()

2013-04-10 Thread Chung-Ju Wu
2013/4/9 Eric Botcazou : >> On behalf of Andes Technology Co., we have signed FSF agreement. >> However, so far I don't have svn write access yet. >> Would you please help to commit this patch? > > Jeff has kindly offered to sponsor you for write access, so you should be able > to install it yourse

Re: [PATCH, combine] Fix host-specific behavior in simplify_compare_const()

2013-04-09 Thread Eric Botcazou
> The attached patch has passed bootstrap/regression test > on x86_64-unknown-linux-gnu with current main trunk. > A plaintext gcc/ChangeLog is as below: > > > 2013-04-06 Chung-Ju Wu > > * combine.c (simplify_compare_const): Use GET_MODE_MASK to filter > out unnecessary bits in the co

Re: [PATCH, combine] Fix host-specific behavior in simplify_compare_const()

2013-04-07 Thread Chung-Ju Wu
2013/4/7 Jeff Law : > On 04/06/2013 01:42 AM, Chung-Ju Wu wrote: >> >> On behalf of Andes Technology Co., we have signed FSF agreement. >> However, so far I don't have svn write access yet. >> Would you please help to commit this patch? >> >> Thanks again for the approval and I really appreciate yo

Re: [PATCH, combine] Fix host-specific behavior in simplify_compare_const()

2013-04-06 Thread Jeff Law
On 04/06/2013 01:42 AM, Chung-Ju Wu wrote: The attached patch has passed bootstrap/regression test on x86_64-unknown-linux-gnu with current main trunk. A plaintext gcc/ChangeLog is as below: 2013-04-06 Chung-Ju Wu * combine.c (simplify_compare_const): Use GET_MODE_MASK to filter

Re: [PATCH, combine] Fix host-specific behavior in simplify_compare_const()

2013-04-06 Thread Chung-Ju Wu
2013/3/7 Chung-Ju Wu : > 2013/3/5 Eric Botcazou : >>> In other words, any 32-bit target with 'need_64bit_hwint=yes' in config.gcc >>> is not able to have benefit from this optimization because it never >>> passes the condition test. >>> >>> >>> My solution is to use GET_MODE_MASK(mode) to filter ou

Re: [PATCH, combine] Fix host-specific behavior in simplify_compare_const()

2013-03-06 Thread Chung-Ju Wu
2013/3/5 Eric Botcazou : >> In other words, any 32-bit target with 'need_64bit_hwint=yes' in config.gcc >> is not able to have benefit from this optimization because it never >> passes the condition test. >> >> >> My solution is to use GET_MODE_MASK(mode) to filter out all bits not >> in target mod

Re: [PATCH, combine] Fix host-specific behavior in simplify_compare_const()

2013-03-05 Thread Eric Botcazou
> In other words, any 32-bit target with 'need_64bit_hwint=yes' in config.gcc > is not able to have benefit from this optimization because it never > passes the condition test. > > > My solution is to use GET_MODE_MASK(mode) to filter out all bits not > in target mode. The following is my patch:

[PATCH, combine] Fix host-specific behavior in simplify_compare_const()

2013-02-27 Thread Chung-Ju Wu
Hi all, I notice the function simplify_compare_const() in gcc/combine.c presents different behavior between 32-bit host machine and 64-bit host machine, causing it to perform different RTL transformation for a 32-bit target. In the function: simplify_compare_const (enum rtx_code code, rtx op0, rt