Re: [PATCH GCC]Look into unnecessary conversion when checking mult_op in get_shiftadd_cost

2015-09-14 Thread Bin.Cheng
On Wed, Sep 2, 2015 at 8:32 PM, Richard Biener wrote: > On Wed, Sep 2, 2015 at 5:50 AM, Bin Cheng wrote: >> Hi, >> When calling get_shiftadd_cost, the mult_op is stripped at caller places. >> We should look into unnecessary conversion in op1 before checking equality, >> otherwise it computes wron

Re: [PATCH GCC]Look into unnecessary conversion when checking mult_op in get_shiftadd_cost

2015-09-02 Thread Richard Biener
On Wed, Sep 2, 2015 at 5:50 AM, Bin Cheng wrote: > Hi, > When calling get_shiftadd_cost, the mult_op is stripped at caller places. > We should look into unnecessary conversion in op1 before checking equality, > otherwise it computes wrong shiftadd cost. This patch picks this small > issue up. > >

[PATCH GCC]Look into unnecessary conversion when checking mult_op in get_shiftadd_cost

2015-09-01 Thread Bin Cheng
Hi, When calling get_shiftadd_cost, the mult_op is stripped at caller places. We should look into unnecessary conversion in op1 before checking equality, otherwise it computes wrong shiftadd cost. This patch picks this small issue up. Bootstrap and test on x86_64 and aarch64 along with other patc