This is the mail archive of the binutils@sourceware.org mailing list for the binutils project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH] gas/config/tc-score7.c: Use symbol_get_frag() instead of direct reference to fix segment fault issue


Firstly, thank you very much for your work (about the 2 patches), and I
shall continue to try to find and provide new patches to binutils (hope
I can finish 1-3 patches per month).

On 06/13/2014 11:10 PM, Nicholas Clifton wrote:
> Hi Chen,
> 
>> Is it patch OK? please help check when you have free time, thanks.
> 
> Approved and applied.  Note - there were similar errors in tc-score.c so
> I fixed them at the same time.
> 

Thank you for your additional work.

>> Originally, I could not find the related Changelog for 'gas', so I did
>> not mark change log for it.
> 
> All patches should have a changelog entry, so I created this for yours:
> 
> gas/ChangeLog
> 2014-06-13  Chen Gang  <gang.chen.5i5j@gmail.com>
> 
>     * config/tc-score7.c: (s7_b32_relax_to_b16): Use symbol_get_frag()
> to access a symbol's
>     frag.
>     * config/tc-score.c (s3_relax_branch_inst16): Likewise.
>     (s3_relax_cmpbranch_inst32): Likewise.
> 

Thanks again, next, I shall always provide ChangLog for any binutils
patches.

> 
> 
> Also - this is a situation where it would have been appropriate to
> create an official bug report on the bugzilla system.
> 

If it is necessary to let me create the bug report, please let me know
(and then, I shall do). Or please help to try to create it on related
bugzilla system.


Thanks.
-- 
Chen Gang

Open, share, and attitude like air, water, and life which God blessed


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]