On 07/05/2012 08:29 AM, Nick Clifton wrote:
Hi Jeff, Hi Alex,

   This may count as an obvious fix, but I felt that I ought to check,
   just in case...  Currently mn10300_encode_section_info() does not call
   default_encode_section_info().  This means that it misses out on
   possibly setting some important flags in the symbol, such as
   SYMBOL_FLAG_FUNCTION, SYMBOL_FLAG_LOCAL or the TLS encoding.

   Fixed by the following patch, and tested on an mn10300-elf toolchain.

   OK for mainline and 4.7/4.6 branches ?

Cheers
   Nick

gcc/ChangeLog
2012-07-05  Nick Clifton  <ni...@redhat.com>

        * config/mn10300/mn10300.c (mn10300_encode_section_info): Call
        default_encode_section_info.
OK. Presumably it doesn't matter in any significant way whether the default_encode_section_info is called early or late in that routine, or if it does for the mn103 you selected the right call point. I spot checked several other ports and there's no clear consensus on that issue.


Jeff

Reply via email to